Fix possible naming conflict in macro.

base/observer_list.h

Index: base/observer_list.h
diff --git a/base/observer_list.h b/base/observer_list.h
index b44e33f56c73f4e2a749436a13be8241518a307d..b3da2e10cbc72e934d81f66d3ef5e60ba33e4dd4 100644
--- a/base/observer_list.h
+++ b/base/observer_list.h
@@ -203,14 +203,14 @@ class ObserverList : public ObserverListBase<ObserverType> {
   }
 };
 
-#define FOR_EACH_OBSERVER(ObserverType, observer_list, func)         \
-  do {                                                               \
-    if ((observer_list).might_have_observers()) {                    \
-      ObserverListBase<ObserverType>::Iterator it(observer_list);    \
-      ObserverType* obs;                                             \
-      while ((obs = it.GetNext()) != NULL)                           \
-        obs->func;                                                   \
-    }                                                                \
+#define FOR_EACH_OBSERVER(ObserverType, observer_list, func)               \
+  do {                                                                     \
+    if ((observer_list).might_have_observers()) {                          \
+      ObserverListBase<ObserverType>::Iterator it_macro(observer_list);    \
+      ObserverType* obs;                                                   \
+      while ((obs = it_macro.GetNext()) != NULL)                           \

[jar (doing other things), 2013/09/23 16:27:15]

This looks good... you might even go bigger, to further reduce the chance that
the name will conflict with callers.  Maybe a name like:

it_inside_observer_macro

Alternatively, you could ask why we needed to make this a macro?  Could we turn
this into a template-method that gets inlined?

+        obs->func;                                                         \
+    }                                                                      \
   } while (0)
 
 #endif  // BASE_OBSERVER_LIST_H__