Add NetworkConnectionHandler class

chromeos/dbus/shill_manager_client.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 CHROMEOS_DBUS_SHILL_MANAGER_CLIENT_H_
 #define CHROMEOS_DBUS_SHILL_MANAGER_CLIENT_H_
 
 #include <string>
 
 #include "base/basictypes.h"
@@ skipping 24 matching lines @@
   typedef ShillClientHelper::StringCallback StringCallback;
   typedef ShillClientHelper::BooleanCallback BooleanCallback;
 
   // Interface for setting up devices, services, and technologies for testing.
   // Accessed through GetTestInterface(), only implemented in the Stub Impl.
   class TestInterface {
    public:
     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,
                                            bool initializing) = 0;
     virtual void AddGeoNetwork(const std::string& technology,
                                const base::DictionaryValue& network) = 0;
 
     // Used to reset all properties; does not notify observers.
     virtual void ClearProperties() = 0;
 
@@ skipping 127 matching lines @@
   // Create() should be used instead.
   ShillManagerClient();
 
  private:
   DISALLOW_COPY_AND_ASSIGN(ShillManagerClient);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROMEOS_DBUS_SHILL_MANAGER_CLIENT_H_