Adding non-tracking mode support to AddressTracker.

net/base/address_tracker_linux.h

 // 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.
 
 #ifndef NET_BASE_ADDRESS_TRACKER_LINUX_H_
 #define NET_BASE_ADDRESS_TRACKER_LINUX_H_
 
 #include <sys/socket.h>  // Needed to include netlink.
 // Mask superfluous definition of |struct net|. This is fixed in Linux 2.6.38.
 #define net net_kernel
 #include <linux/rtnetlink.h>
 #undef net
 
 #include <map>
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/compiler_specific.h"
 #include "base/containers/hash_tables.h"
 #include "base/message_loop/message_loop.h"
 #include "base/synchronization/condition_variable.h"
 #include "base/synchronization/lock.h"
+#include "base/threading/thread_checker.h"
 #include "net/base/net_util.h"
 #include "net/base/network_change_notifier.h"
 
 namespace net {
 namespace internal {
 
 // Keeps track of network interface addresses using rtnetlink. Used by
 // NetworkChangeNotifier to provide signals to registered IPAddressObservers.
 class NET_EXPORT_PRIVATE AddressTrackerLinux :
     public base::MessageLoopForIO::Watcher {
  public:
   typedef std::map<IPAddressNumber, struct ifaddrmsg> AddressMap;
 
-  // Will run |address_callback| when the AddressMap changes, |link_callback|
-  // when the list of online links changes, and |tunnel_callback| when the list
-  // of online tunnels changes.
+  // Non-tracking version constructor: it takes a snapshot of current
+  // system configuration once Init() returns. The configuration is

[pauljensen, 2014/09/19 19:25:39]

This is incorrect, the snapshot is not taken when Init() returns, it's take
before that.  How about "it takes a snapshot of the current system
configuration.  Once Init() returns, the configuration is available through
GetOnlineLinks() and GetAddressMap()."

[guoweis2, 2014/09/19 22:12:55]

Done.

+  // available through GetOnlineLinks() and GetAddressMap().
+  AddressTrackerLinux();
+
+  // Tracking version constructor: it will run |address_callback| when
+  // the AddressMap changes, |link_callback| when the list of online
+  // links changes, and |tunnel_callback| when the list of online
+  // tunnels changes.
   AddressTrackerLinux(const base::Closure& address_callback,
                       const base::Closure& link_callback,
                       const base::Closure& tunnel_callback);
   virtual ~AddressTrackerLinux();
 
-  // Starts watching system configuration for changes. The current thread must
-  // have a MessageLoopForIO.
+  // In tracking mode, it starts watching system configuration for

[pauljensen, 2014/09/19 19:25:40]

system->the system

[guoweis2, 2014/09/19 22:12:55]

Done.

+  // changes. The current thread must have a MessageLoopForIO. For
+  // non-tracking mode, the |AddressMap| and |online_links_| will be a
+  // one time snapshot of current system configuration after Init()
+  // returns.

[pauljensen, 2014/09/19 19:25:40]

|AddressMap| is a type but |online_links_| is a private variable...  How about
"In non-tracking mode, once Init() returns, a snapshot of the system
configuration is available through GetOnlineLinks() and GetAddressMap()."

[guoweis2, 2014/09/19 22:12:55]

Done.

   void Init();
 
   AddressMap GetAddressMap() const;
+  base::hash_set<int> GetOnlineLinks() const;

[pauljensen, 2014/09/19 19:25:39]

This needs a comment, maybe "Returns set of interface indicies for online
interfaces."

[guoweis2, 2014/09/19 22:12:55]

Done.

 
   // Implementation of NetworkChangeNotifierLinux::GetCurrentConnectionType().
   // Safe to call from any thread, but will block until Init() has completed.
   NetworkChangeNotifier::ConnectionType GetCurrentConnectionType();
 
  private:
   friend class AddressTrackerLinuxTest;
 
+  // In tracking mode, holds |lock| while alive. In non-tracking mode,
+  // enforces single-threaded access.
+  class AddressTrackerAutoLock {
+   public:
+    AddressTrackerAutoLock(const AddressTrackerLinux& tracker,
+                           base::Lock& lock);
+    ~AddressTrackerAutoLock();
+
+   private:
+    const AddressTrackerLinux& tracker_;
+    base::Lock& lock_;
+    DISALLOW_COPY_AND_ASSIGN(AddressTrackerAutoLock);
+  };
+
   // A function that returns the name of an interface given the interface index
   // in |interface_index|.
   typedef const char* (*GetInterfaceNameFunction)(int interface_index);
 
   // Sets |*address_changed| to indicate whether |address_map_| changed and
   // sets |*link_changed| to indicate if |online_links_| changed and sets
   // |*tunnel_changed| to indicate if |online_links_| changed with regards to a
   // tunnel interface while reading messages from |netlink_fd_|.
   void ReadMessages(bool* address_changed,
                     bool* link_changed,
@@ skipping 31 matching lines @@
   base::Closure link_callback_;
   base::Closure tunnel_callback_;
 
   int netlink_fd_;
   base::MessageLoopForIO::FileDescriptorWatcher watcher_;
 
   mutable base::Lock address_map_lock_;
   AddressMap address_map_;
 
   // Set of interface indices for links that are currently online.
+  mutable base::Lock online_links_lock_;
   base::hash_set<int> online_links_;
 
   base::Lock is_offline_lock_;
   bool is_offline_;
   bool is_offline_initialized_;
   base::ConditionVariable is_offline_initialized_cv_;
+  bool tracking_;
+
+  // A |ThreadChecker| used to verify single-threaded access in

[pauljensen, 2014/09/19 19:25:40]

Remove "A |ThreadChecker|" as this is redundant with the code.

[guoweis2, 2014/09/19 22:12:55]

Done.

+  // non-tracking mode.
+  base::ThreadChecker thread_checker_;
 };
 
 }  // namespace internal
 }  // namespace net
 
 #endif  // NET_BASE_ADDRESS_TRACKER_LINUX_H_