Remove FOR_EACH_OBSERVER from arc/ since it's being deprecated

components/arc/arc_bridge_service.cc

 // Copyright 2015 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 "components/arc/arc_bridge_service.h"
 
 #include <utility>
 
 #include "base/command_line.h"
 #include "base/feature_list.h"
@@ skipping 55 matching lines @@
 void ArcBridgeService::RemoveObserver(Observer* observer) {
   DCHECK(CalledOnValidThread());
   observer_list_.RemoveObserver(observer);
 }
 
 void ArcBridgeService::SetState(State state) {
   DCHECK(CalledOnValidThread());
   DCHECK_NE(state_, state);
   state_ = state;
   VLOG(2) << "State: " << static_cast<uint32_t>(state_);
-  if (state_ == State::READY)
-    FOR_EACH_OBSERVER(Observer, observer_list(), OnBridgeReady());
-  else if (state == State::STOPPED)
-    FOR_EACH_OBSERVER(Observer, observer_list(), OnBridgeStopped(stop_reason_));
+  if (state_ == State::READY) {
+    for (auto& observer : observer_list())
+      observer.OnBridgeReady();
+  } else if (state == State::STOPPED) {
+    for (auto& observer : observer_list())

[Yusuke Sato, 2016/10/12 18:30:35]

Added <LF> since line 80-81 does not fit in one line. Also, one-line for does
not seem to be very popular in Chromium.

+      observer.OnBridgeStopped(stop_reason_);
+  }
 }
 
 void ArcBridgeService::SetStopReason(StopReason stop_reason) {
   DCHECK(CalledOnValidThread());
   stop_reason_ = stop_reason;
 }
 
 bool ArcBridgeService::CalledOnValidThread() {
   return thread_checker_.CalledOnValidThread();
 }
@@ skipping 10 matching lines @@
     CASE_IMPL(LOW_DISK_SPACE);
     CASE_IMPL(CRASH);
 #undef CASE_IMPL
   }
 
   // In case of unexpected value, output the int value.
   return os << "StopReason(" << static_cast<int>(reason) << ")";
 }
 
 }  // namespace arc