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

Unified Diff: chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.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
Index: chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.h
diff --git a/chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.h b/chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.h
index bf0fc7bdbea7dc02218468fa47418634115a3862..e4845c1bfa25f40eb2c1610c3fff3068004d66d6 100644
--- a/chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.h
+++ b/chrome/browser/extensions/api/declarative_content/declarative_content_css_condition_tracker.h
@@ -24,13 +24,6 @@ namespace base {
class Value;
}
-namespace content {
-struct FrameNavigateParams;
-struct LoadCommittedDetails;
-class RenderProcessHost;
-class WebContents;
-}
-
namespace extensions {
class Extension;
@@ -86,8 +79,7 @@ class DeclarativeContentCssConditionTracker
void TrackForWebContents(content::WebContents* contents) override;
void OnWebContentsNavigation(
content::WebContents* contents,
- const content::LoadCommittedDetails& details,
- const content::FrameNavigateParams& params) override;
+ content::NavigationHandle* navigation_handle) override;
bool EvaluatePredicate(const ContentPredicate* predicate,
content::WebContents* tab) const override;
@@ -106,8 +98,7 @@ class DeclarativeContentCssConditionTracker
const WebContentsDestroyedCallback& web_contents_destroyed);
~PerWebContentsTracker() override;
- void OnWebContentsNavigation(const content::LoadCommittedDetails& details,
- const content::FrameNavigateParams& params);
+ void OnWebContentsNavigation(content::NavigationHandle* navigation_handle);
const base::hash_set<std::string>& matching_css_selectors() const {
return matching_css_selectors_;

Powered by Google App Engine
This is Rietveld 408576698