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

Unified Diff: extensions/browser/api/declarative_content/content_rules_registry.h

Issue 2668003004: Convert extensions::TabHelper to use the new navigation callbacks. (Closed)
Patch Set: update Created 3 years, 11 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
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/api/declarative_content/content_rules_registry.h
diff --git a/extensions/browser/api/declarative_content/content_rules_registry.h b/extensions/browser/api/declarative_content/content_rules_registry.h
index 239687fa7b72ba7014f7bc8d928269538dfbe537..2c6716040522e145aef766635121ef59e4751b6c 100644
--- a/extensions/browser/api/declarative_content/content_rules_registry.h
+++ b/extensions/browser/api/declarative_content/content_rules_registry.h
@@ -13,9 +13,8 @@
namespace content {
class BrowserContext;
+class NavigationHandle;
class WebContents;
-struct FrameNavigateParams;
-struct LoadCommittedDetails;
}
namespace extensions {
@@ -48,10 +47,11 @@ class ContentRulesRegistry : public RulesRegistry {
content::WebContents* contents) = 0;
// Applies all content rules given that a tab was just navigated.
- virtual void DidNavigateMainFrame(
+ // This corresponds to the notification of the same name in
+ // content::WebContentsObserver.
+ virtual void DidFinishNavigation(
content::WebContents* tab,
- const content::LoadCommittedDetails& details,
- const content::FrameNavigateParams& params) = 0;
+ content::NavigationHandle* navigation_handle) = 0;
protected:
~ContentRulesRegistry() override {}
« no previous file with comments | « chrome/browser/extensions/tab_helper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698