Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1549)

Unified Diff: chrome/browser/extensions/active_tab_permission_granter.cc

Issue 2716493004: Renamed NavigationHandle::IsSamePage to NavigationHandle::IsSameDocument (Closed)
Patch Set: Rebased Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/extensions/active_tab_permission_granter.cc
diff --git a/chrome/browser/extensions/active_tab_permission_granter.cc b/chrome/browser/extensions/active_tab_permission_granter.cc
index 8d43372b15494ee1a04351ee605195434883d02a..b24436d801aaddb32770ea05433c1cd21e6762cb 100644
--- a/chrome/browser/extensions/active_tab_permission_granter.cc
+++ b/chrome/browser/extensions/active_tab_permission_granter.cc
@@ -144,7 +144,7 @@ void ActiveTabPermissionGranter::DidFinishNavigation(
// Important: sub-frames don't get granted!
if (!navigation_handle->IsInMainFrame() ||
!navigation_handle->HasCommitted() ||
- navigation_handle->IsSamePage()) {
+ navigation_handle->IsSameDocument()) {
return;
}

Powered by Google App Engine
This is Rietveld 408576698