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

Unified Diff: extensions/browser/script_executor.cc

Issue 2424593002: Reduce FOR_EACH_OBSERVER usage in extensions (Closed)
Patch Set: script_observers_ Created 4 years, 2 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 | « extensions/browser/process_manager.cc ('k') | extensions/browser/user_script_loader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: extensions/browser/script_executor.cc
diff --git a/extensions/browser/script_executor.cc b/extensions/browser/script_executor.cc
index 105889c3121ab97e6a23ad4c9d5f370df28808ad..db2c02da3031784a258f5e4abb7ac972cee8d177 100644
--- a/extensions/browser/script_executor.cc
+++ b/extensions/browser/script_executor.cc
@@ -168,9 +168,8 @@ class Handler : public content::WebContentsObserver {
host_id_.type() == HostID::EXTENSIONS) {
ScriptExecutionObserver::ExecutingScriptsMap id_map;
id_map[host_id_.id()] = std::set<std::string>();
- FOR_EACH_OBSERVER(
- ScriptExecutionObserver, *script_observers_,
- OnScriptsExecuted(web_contents(), id_map, root_frame_url_));
+ for (auto& observer : *script_observers_)
+ observer.OnScriptsExecuted(web_contents(), id_map, root_frame_url_);
}
if (!callback_.is_null())
« no previous file with comments | « extensions/browser/process_manager.cc ('k') | extensions/browser/user_script_loader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698