Reduce FOR_EACH_OBSERVER macro usage in chrome/browser

chrome/browser/background/background_contents.cc

 // Copyright (c) 2012 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 "chrome/browser/background/background_contents.h"
 
 #include <utility>
 
 #include "base/profiler/scoped_tracker.h"
 #include "chrome/browser/background/background_contents_service.h"
@@ skipping 81 matching lines @@
 
   // Unregister for any notifications before notifying observers that we are
   // going away - this prevents any re-entrancy due to chained notifications
   // (http://crbug.com/237781).
   registrar_.RemoveAll();
 
   content::NotificationService::current()->Notify(
       chrome::NOTIFICATION_BACKGROUND_CONTENTS_DELETED,
       content::Source<Profile>(profile_),
       content::Details<BackgroundContents>(this));
-  FOR_EACH_OBSERVER(extensions::DeferredStartRenderHostObserver,
-                    deferred_start_render_host_observer_list_,
-                    OnDeferredStartRenderHostDestroyed(this));
+  for (auto& observer : deferred_start_render_host_observer_list_)
+    observer.OnDeferredStartRenderHostDestroyed(this);
 
   extension_host_delegate_->GetExtensionHostQueue()->Remove(this);
 }
 
 const GURL& BackgroundContents::GetURL() const {
   return web_contents_.get() ? web_contents_->GetURL() : GURL::EmptyGURL();
 }
 
 void BackgroundContents::CreateRenderViewSoon(const GURL& url) {
   initial_url_ = url;
@@ skipping 52 matching lines @@
   // Our RenderView went away, so we should go away also, so killing the process
   // via the TaskManager doesn't permanently leave a BackgroundContents hanging
   // around the system, blocking future instances from being created
   // <http://crbug.com/65189>.
   delete this;
 }
 
 void BackgroundContents::DidStartLoading() {
   // BackgroundContents only loads once, so this can only be the first time it
   // has started loading.
-  FOR_EACH_OBSERVER(extensions::DeferredStartRenderHostObserver,
-                    deferred_start_render_host_observer_list_,
-                    OnDeferredStartRenderHostDidStartFirstLoad(this));
+  for (auto& observer : deferred_start_render_host_observer_list_)
+    observer.OnDeferredStartRenderHostDidStartFirstLoad(this);
 }
 
 void BackgroundContents::DidStopLoading() {
   // BackgroundContents only loads once, so this can only be the first time
   // it has stopped loading.
-  FOR_EACH_OBSERVER(extensions::DeferredStartRenderHostObserver,
-                    deferred_start_render_host_observer_list_,
-                    OnDeferredStartRenderHostDidStopFirstLoad(this));
+  for (auto& observer : deferred_start_render_host_observer_list_)
+    observer.OnDeferredStartRenderHostDidStopFirstLoad(this);
 }
 
 void BackgroundContents::Observe(int type,
                                  const content::NotificationSource& source,
                                  const content::NotificationDetails& details) {
   // TODO(rafaelw): Implement pagegroup ref-counting so that non-persistent
   // background pages are closed when the last referencing frame is closed.
   switch (type) {
     case chrome::NOTIFICATION_PROFILE_DESTROYED:
     case chrome::NOTIFICATION_APP_TERMINATING: {
@@ skipping 18 matching lines @@
 
 void BackgroundContents::AddDeferredStartRenderHostObserver(
     extensions::DeferredStartRenderHostObserver* observer) {
   deferred_start_render_host_observer_list_.AddObserver(observer);
 }
 
 void BackgroundContents::RemoveDeferredStartRenderHostObserver(
     extensions::DeferredStartRenderHostObserver* observer) {
   deferred_start_render_host_observer_list_.RemoveObserver(observer);
 }