Convert extensions::TabHelper to use the new navigation callbacks.

extensions/browser/api/declarative_content/content_rules_registry.h

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..6bc996836260aa31ed9b662f9e28bf81f7b3db88 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,9 @@ class ContentRulesRegistry : public RulesRegistry {
       content::WebContents* contents) = 0;
 
   // Applies all content rules given that a tab was just navigated.

[Devlin, 2017/02/01 23:18:22]

nit: while you're here, mind adding something like:
// This corresponds to the notification of the same name in
// content::WebContentsObserver.
?

[jam, 2017/02/01 23:48:49]

Done.

-  virtual void DidNavigateMainFrame(
+  virtual void DidFinishNavigation(
       content::WebContents* tab,
-      const content::LoadCommittedDetails& details,
-      const content::FrameNavigateParams& params) = 0;
+      content::NavigationHandle* navigation_handle) = 0;
 
  protected:
   ~ContentRulesRegistry() override {}