Reduce FOR_EACH_OBSERVER usage in extensions

extensions/browser/user_script_loader.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "extensions/browser/user_script_loader.h"
 
 #include <stddef.h>
 
 #include <set>
 #include <string>
@@ skipping 152 matching lines @@
       pending_load_(false),
       browser_context_(browser_context),
       host_id_(host_id),
       weak_factory_(this) {
   registrar_.Add(this,
                  content::NOTIFICATION_RENDERER_PROCESS_CREATED,
                  content::NotificationService::AllBrowserContextsAndSources());
 }
 
 UserScriptLoader::~UserScriptLoader() {
-  FOR_EACH_OBSERVER(Observer, observers_, OnUserScriptLoaderDestroyed(this));
+  for (auto& observer : observers_)
+    observer.OnUserScriptLoaderDestroyed(this);
 }
 
 void UserScriptLoader::AddScripts(std::unique_ptr<UserScriptList> scripts) {
 #if DCHECK_IS_ON()
   // |scripts| with non-unique IDs will work, but that would indicate we are
   // doing something wrong somewhere, so DCHECK that.
   DCHECK(AreScriptsUnique(*scripts))
       << "AddScripts() expects scripts with unique IDs.";
 #endif  // DCHECK_IS_ON()
   for (std::unique_ptr<UserScript>& user_script : *scripts) {
@@ skipping 203 matching lines @@
        !i.IsAtEnd(); i.Advance()) {
     SendUpdate(i.GetCurrentValue(), shared_memory_.get(), changed_hosts_);
   }
   changed_hosts_.clear();
 
   // TODO(hanxi): Remove the NOTIFICATION_USER_SCRIPTS_UPDATED.
   content::NotificationService::current()->Notify(
       extensions::NOTIFICATION_USER_SCRIPTS_UPDATED,
       content::Source<BrowserContext>(browser_context_),
       content::Details<base::SharedMemory>(shared_memory_.get()));
-  FOR_EACH_OBSERVER(Observer, observers_, OnScriptsLoaded(this));
+  for (auto& observer : observers_)
+    observer.OnScriptsLoaded(this);
 }
 
 void UserScriptLoader::SendUpdate(content::RenderProcessHost* process,
                                   base::SharedMemory* shared_memory,
                                   const std::set<HostID>& changed_hosts) {
   // Don't allow injection of non-whitelisted extensions' content scripts
   // into <webview>.
   bool whitelisted_only = process->IsForGuestsOnly() && host_id().id().empty();
 
   // Make sure we only send user scripts to processes in our browser_context.
@@ skipping 11 matching lines @@
   if (!shared_memory->ShareToProcess(handle, &handle_for_process))
     return;  // This can legitimately fail if the renderer asserts at startup.
 
   if (base::SharedMemory::IsHandleValid(handle_for_process)) {
     process->Send(new ExtensionMsg_UpdateUserScripts(
         handle_for_process, host_id(), changed_hosts, whitelisted_only));
   }
 }
 
 }  // namespace extensions