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

Unified Diff: trunk/src/chrome/browser/extensions/tab_helper.cc

Issue 16109003: Revert 202751 "Change RulesRegistryService to use ProfileKeyedAPI." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 7 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: trunk/src/chrome/browser/extensions/tab_helper.cc
===================================================================
--- trunk/src/chrome/browser/extensions/tab_helper.cc (revision 202762)
+++ trunk/src/chrome/browser/extensions/tab_helper.cc (working copy)
@@ -85,6 +85,10 @@
pending_web_app_action_(NONE),
script_executor_(new ScriptExecutor(web_contents,
&script_execution_observers_)),
+ rules_registry_service_(
+ ExtensionSystem::Get(
+ Profile::FromBrowserContext(web_contents->GetBrowserContext()))->
+ rules_registry_service()),
image_loader_ptr_factory_(this) {
// The ActiveTabPermissionManager requires a session ID; ensure this
// WebContents has one.
@@ -190,9 +194,9 @@
const content::LoadCommittedDetails& details,
const content::FrameNavigateParams& params) {
#if defined(ENABLE_EXTENSIONS)
- if (ExtensionSystem::Get(profile_)->extension_service()) {
- RulesRegistryService::Get(profile_)->content_rules_registry()->
- DidNavigateMainFrame(web_contents(), details, params);
+ if (rules_registry_service_) {
+ rules_registry_service_->content_rules_registry()->DidNavigateMainFrame(
+ web_contents(), details, params);
}
#endif // defined(ENABLE_EXTENSIONS)
@@ -341,8 +345,8 @@
void TabHelper::OnWatchedPageChange(
const std::vector<std::string>& css_selectors) {
#if defined(ENABLE_EXTENSIONS)
- if (ExtensionSystem::Get(profile_)->extension_service()) {
- RulesRegistryService::Get(profile_)->content_rules_registry()->Apply(
+ if (rules_registry_service_) {
+ rules_registry_service_->content_rules_registry()->Apply(
web_contents(), css_selectors);
}
#endif // defined(ENABLE_EXTENSIONS)
Property changes on: trunk/src/chrome/browser/extensions/tab_helper.cc
___________________________________________________________________
Deleted: svn:mergeinfo
« no previous file with comments | « trunk/src/chrome/browser/extensions/tab_helper.h ('k') | trunk/src/chrome/browser/extensions/test_extension_system.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698