Remove usage of FOR_EACH_OBSERVER macro in components/sync

components/sync/test/fake_server/fake_server.cc

Index: components/sync/test/fake_server/fake_server.cc
diff --git a/components/sync/test/fake_server/fake_server.cc b/components/sync/test/fake_server/fake_server.cc
index a24a22095eaf10432d0ddea1be71cfa0b78a77b8..33869b7828e98d29f5f6a05e9d9b772f83176f2c 100644
--- a/components/sync/test/fake_server/fake_server.cc
+++ b/components/sync/test/fake_server/fake_server.cc
@@ -509,8 +509,8 @@ bool FakeServer::HandleCommitRequest(const sync_pb::CommitMessage& commit,
     committed_model_types.Put(iter->second->model_type());
   }
 
-  FOR_EACH_OBSERVER(Observer, observers_,
-                    OnCommit(invalidator_client_id, committed_model_types));
+  for (auto& observer : observers_)
+    observer.OnCommit(invalidator_client_id, committed_model_types);
   return true;

[maxbogue, 2016/10/17 16:54:58]

This sort of thing gives me anxiety, and is why I'd prefer for loops and if
statements to always have their {}'s, but apparently I'm in a minority with that
sentiment.

 }