Reduce usage of FOR_EACH_OBSERVER macro in content/browser

content/browser/devtools/service_worker_devtools_manager.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 "content/browser/devtools/service_worker_devtools_manager.h"
 
 #include "content/browser/devtools/devtools_agent_host_impl.h"
 #include "content/browser/devtools/service_worker_devtools_agent_host.h"
 #include "content/public/browser/browser_thread.h"
 #include "content/public/browser/render_process_host.h"
@@ skipping 71 matching lines @@
     const ServiceWorkerIdentifier& service_worker_id,
     bool is_installed_version) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   const WorkerId id(worker_process_id, worker_route_id);
   AgentHostMap::iterator it = FindExistingWorkerAgentHost(service_worker_id);
   if (it == workers_.end()) {
     scoped_refptr<ServiceWorkerDevToolsAgentHost> host =
         new ServiceWorkerDevToolsAgentHost(id, service_worker_id,
                                            is_installed_version);
     workers_[id] = host.get();
-    FOR_EACH_OBSERVER(Observer, observer_list_, WorkerCreated(host.get()));
+    for (auto& observer : observer_list_)
+      observer.WorkerCreated(host.get());
     if (debug_service_worker_on_start_)
         host->PauseForDebugOnStart();
     return host->IsPausedForDebugOnStart();
   }
 
   // Worker was restarted.
   ServiceWorkerDevToolsAgentHost* agent_host = it->second;
   agent_host->WorkerRestarted(id);
   workers_.erase(it);
   workers_[id] = agent_host;
 
   return agent_host->IsAttached();
 }
 
 void ServiceWorkerDevToolsManager::WorkerReadyForInspection(
     int worker_process_id,
     int worker_route_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   const WorkerId id(worker_process_id, worker_route_id);
   AgentHostMap::iterator it = workers_.find(id);
   if (it == workers_.end())
     return;
   scoped_refptr<ServiceWorkerDevToolsAgentHost> host = it->second;
   host->WorkerReadyForInspection();
-  FOR_EACH_OBSERVER(Observer, observer_list_,
-                    WorkerReadyForInspection(host.get()));
+  for (auto& observer : observer_list_)
+    observer.WorkerReadyForInspection(host.get());
 
   // Then bring up UI for the ones not picked by other clients.
   if (host->IsPausedForDebugOnStart() && !host->IsAttached())
     static_cast<DevToolsAgentHostImpl*>(host.get())->Inspect();
 }
 
 void ServiceWorkerDevToolsManager::WorkerVersionInstalled(int worker_process_id,
                                                           int worker_route_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   const WorkerId id(worker_process_id, worker_route_id);
   AgentHostMap::iterator it = workers_.find(id);
   if (it == workers_.end())
     return;
   scoped_refptr<ServiceWorkerDevToolsAgentHost> host = it->second;
   host->WorkerVersionInstalled();
-  FOR_EACH_OBSERVER(Observer, observer_list_,
-                    WorkerVersionInstalled(host.get()));
+  for (auto& observer : observer_list_)
+    observer.WorkerVersionInstalled(host.get());
 }
 
 void ServiceWorkerDevToolsManager::WorkerVersionDoomed(int worker_process_id,
                                                        int worker_route_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   const WorkerId id(worker_process_id, worker_route_id);
   AgentHostMap::iterator it = workers_.find(id);
   if (it == workers_.end())
     return;
   scoped_refptr<ServiceWorkerDevToolsAgentHost> host = it->second;
   host->WorkerVersionDoomed();
-  FOR_EACH_OBSERVER(Observer, observer_list_, WorkerVersionDoomed(host.get()));
+  for (auto& observer : observer_list_)
+    observer.WorkerVersionDoomed(host.get());
 }
 
 void ServiceWorkerDevToolsManager::WorkerDestroyed(int worker_process_id,
                                                    int worker_route_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   const WorkerId id(worker_process_id, worker_route_id);
   AgentHostMap::iterator it = workers_.find(id);
   if (it == workers_.end())
     return;
   scoped_refptr<WorkerDevToolsAgentHost> agent_host(it->second);
   agent_host->WorkerDestroyed();
-  FOR_EACH_OBSERVER(Observer, observer_list_, WorkerDestroyed(it->second));
+  for (auto& observer : observer_list_)
+    observer.WorkerDestroyed(it->second);
 }
 
 void ServiceWorkerDevToolsManager::RemoveInspectedWorkerData(WorkerId id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   workers_.erase(id);
 }
 
 void ServiceWorkerDevToolsManager::AddObserver(Observer* observer) {
   observer_list_.AddObserver(observer);
 }
@@ skipping 23 matching lines @@
       break;
   }
   return it;
 }
 
 void ServiceWorkerDevToolsManager::ResetForTesting() {
   workers_.clear();
 }
 
 }  // namespace content