Reduce FOR_EACH_OBSERVER usage in ppapi

ppapi/proxy/host_dispatcher.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 "ppapi/proxy/host_dispatcher.h"
 
 #include <stddef.h>
 
 #include "base/logging.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 138 matching lines @@
     // Don't allow sending sync messages during module shutdown. Seee the "else"
     // block below for why.
     CHECK(!PP_ToBool(ppb_proxy()->IsInModuleDestructor(pp_module())));
 
     // Prevent the dispatcher from going away during sync calls. Scenarios
     // where this could happen include a Send for a sync message which while
     // waiting for the reply, dispatches an incoming ExecuteScript call which
     // destroys the plugin module and in turn the dispatcher.
     ScopedModuleReference scoped_ref(this);
 
-    FOR_EACH_OBSERVER(SyncMessageStatusObserver, sync_status_observer_list_,
-                      BeginBlockOnSyncMessage());
+    for (auto& observer : sync_status_observer_list_)
+      observer.BeginBlockOnSyncMessage();
     bool result = Dispatcher::Send(msg);
-    FOR_EACH_OBSERVER(SyncMessageStatusObserver, sync_status_observer_list_,
-                      EndBlockOnSyncMessage());
+    for (auto& observer : sync_status_observer_list_)
+      observer.EndBlockOnSyncMessage();
 
     return result;
   } else {
     // We don't want to have a scoped ref for async message cases since since
     // async messages are sent during module desruction. In this case, the
     // module will have a 0 refcount and addrefing and releasing it will
     // reenter the destructor and it will crash.
     return Dispatcher::Send(msg);
   }
 }
@@ skipping 109 matching lines @@
   }
 }
 
 ScopedModuleReference::~ScopedModuleReference() {
   if (dispatcher_)
     dispatcher_->ppb_proxy()->ReleaseModule(dispatcher_->pp_module());
 }
 
 }  // namespace proxy
 }  // namespace ppapi