Add NetworkConnectionHandler class

chromeos/dbus/shill_manager_client.h

Index: chromeos/dbus/shill_manager_client.h
diff --git a/chromeos/dbus/shill_manager_client.h b/chromeos/dbus/shill_manager_client.h
index 80116bde1c307a40e8d5efd84a422a2e27721150..3e04d29598fcdd6f8037aebe2686feee7285e63a 100644
--- a/chromeos/dbus/shill_manager_client.h
+++ b/chromeos/dbus/shill_manager_client.h
@@ -42,13 +42,14 @@ class CHROMEOS_EXPORT ShillManagerClient {
     virtual void AddDevice(const std::string& device_path) = 0;
     virtual void RemoveDevice(const std::string& device_path) = 0;
     virtual void ClearDevices() = 0;
-    virtual void AddService(const std::string& service_path,
-                            bool add_to_watch_list) = 0;
-    virtual void AddServiceAtIndex(const std::string& service_path,
-                                   size_t index,
+    // Add/Remove/Clear Service should only be called from ShillServiceClient.

[pneubeck (no reviews), 2013/05/07 08:46:39]

nit: Group the functions more clearly (newline before/after) and update the
comment to Add/Remove/Clear ManagerService (?). What about the
MoveServiceToIndex in between?

[stevenjb, 2013/05/08 01:57:22]

Reordered and moved.

+    virtual void AddManagerService(const std::string& service_path,
                                    bool add_to_watch_list) = 0;
-    virtual void RemoveService(const std::string& service_path) = 0;
-    virtual void ClearServices() = 0;
+    virtual void MoveServiceToIndex(const std::string& service_path,
+                                    size_t index,
+                                    bool add_to_watch_list) = 0;
+    virtual void RemoveManagerService(const std::string& service_path) = 0;
+    virtual void ClearManagerServices() = 0;
     virtual void AddTechnology(const std::string& type, bool enabled) = 0;
     virtual void RemoveTechnology(const std::string& type) = 0;
     virtual void SetTechnologyInitializing(const std::string& type,