Massively simplify the NetworkChangeNotifier infrastructure:...

base/observer_list_threadsafe.h

 // Copyright (c) 2010 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.
 
 #ifndef BASE_OBSERVER_LIST_THREADSAFE_H_
 #define BASE_OBSERVER_LIST_THREADSAFE_H_
 
 #include <vector>
 #include <algorithm>
 
@@ skipping 10 matching lines @@
 // OVERVIEW:
 //
 //   A thread-safe container for a list of observers.
 //   This is similar to the observer_list (see observer_list.h), but it
 //   is more robust for multi-threaded situations.
 //
 //   The following use cases are supported:
 //    * Observers can register for notifications from any thread.
 //      Callbacks to the observer will occur on the same thread where
 //      the observer initially called AddObserver() from.
-//    * Any thread may trigger a notification via NOTIFY_OBSERVERS.
+//    * Any thread may trigger a notification via Notify().
 //    * Observers can remove themselves from the observer list inside
 //      of a callback.
 //    * If one thread is notifying observers concurrently with an observer
 //      removing itself from the observer list, the notifications will
 //      be silently dropped.
 //
 //   The drawback of the threadsafe observer list is that notifications
 //   are not as real-time as the non-threadsafe version of this class.
 //   Notifications will always be done via PostTask() to another thread,
 //   whereas with the non-thread-safe observer_list, notifications happen
@@ skipping 150 matching lines @@
   typedef std::map<MessageLoop*, ObserverList<ObserverType>*> ObserversListMap;
 
   // These are marked mutable to facilitate having NotifyAll be const.
   Lock list_lock_;  // Protects the observer_lists_.
   ObserversListMap observer_lists_;
 
   DISALLOW_COPY_AND_ASSIGN(ObserverListThreadSafe);
 };
 
 #endif  // BASE_OBSERVER_LIST_THREADSAFE_H_