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

Unified Diff: chrome/browser/extensions/api/extension_action/extension_action_api.cc

Issue 270153004: Introduce ActiveScriptController; track active extension scripts (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 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: chrome/browser/extensions/api/extension_action/extension_action_api.cc
diff --git a/chrome/browser/extensions/api/extension_action/extension_action_api.cc b/chrome/browser/extensions/api/extension_action/extension_action_api.cc
index 3b552b3cd30ea21b014c1c5073317db540926b07..70e6bbea3f4d5f1e79140b77b413dcfbc42b4714 100644
--- a/chrome/browser/extensions/api/extension_action/extension_action_api.cc
+++ b/chrome/browser/extensions/api/extension_action/extension_action_api.cc
@@ -445,7 +445,7 @@ void ExtensionActionStorageManager::OnExtensionLoaded(
AsWeakPtr(),
extension->id()));
}
-};
+}
void ExtensionActionStorageManager::Observe(
int type,
@@ -632,8 +632,7 @@ void ExtensionActionFunction::NotifyBrowserActionChange() {
}
void ExtensionActionFunction::NotifyLocationBarChange() {
- TabHelper::FromWebContents(contents_)->
- location_bar_controller()->NotifyChange();
+ LocationBarController::NotifyChange(contents_);
}
void ExtensionActionFunction::NotifySystemIndicatorChange() {
@@ -909,8 +908,7 @@ bool PageActionsFunction::SetPageActionEnabled(bool enable) {
// Set visibility and broadcast notifications that the UI should be updated.
page_action->SetIsVisible(tab_id, enable);
page_action->SetTitle(tab_id, title);
- extensions::TabHelper::FromWebContents(contents)->
- location_bar_controller()->NotifyChange();
+ extensions::LocationBarController::NotifyChange(contents);
return true;
}
« no previous file with comments | « chrome/browser/extensions/activity_log/uma_policy_unittest.cc ('k') | chrome/browser/extensions/location_bar_controller.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698