Add chromeos::NetworkStateManager

chromeos/dbus/shill_service_client.cc

 // 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.
 
 #include "chromeos/dbus/shill_service_client.h"
 
 #include "base/bind.h"
 #include "base/chromeos/chromeos_version.h"
 #include "base/message_loop.h"
 #include "base/stl_util.h"
@@ skipping 180 matching lines @@
   }
 
   virtual ~ShillServiceClientStubImpl() {
   }
 
   // ShillServiceClient overrides.
 
   virtual void AddPropertyChangedObserver(
       const dbus::ObjectPath& service_path,
       ShillPropertyChangedObserver* observer) OVERRIDE {
     observer_list_.AddObserver(observer);

[hashimoto, 2012/11/05 04:03:03]

You might want to prepare observer lists for each service_path.
Please see my comment in shill_device_client.cc

[stevenjb, 2012/11/05 22:42:35]

Done.

   }
 
   virtual void RemovePropertyChangedObserver(
       const dbus::ObjectPath& service_path,
       ShillPropertyChangedObserver* observer) OVERRIDE {
     observer_list_.RemoveObserver(observer);
   }
 
   virtual void GetProperties(const dbus::ObjectPath& service_path,
                              const DictionaryValueCallback& callback) OVERRIDE {
@@ skipping 72 matching lines @@
 
   virtual ShillServiceClient::TestInterface* GetTestInterface() OVERRIDE {
     return this;
   }
 
   // ShillServiceClient::TestInterface overrides.
 
   virtual void AddService(const std::string& service_path,
                           const std::string& name,
                           const std::string& type,
-                          const std::string& state,
-                          const std::string& security) OVERRIDE {
+                          const std::string& state) OVERRIDE {
     base::DictionaryValue* properties = GetServiceProperties(service_path);
     properties->SetWithoutPathExpansion(
         flimflam::kSSIDProperty,
         base::Value::CreateStringValue(service_path));
     properties->SetWithoutPathExpansion(
         flimflam::kNameProperty,
         base::Value::CreateStringValue(name));
     properties->SetWithoutPathExpansion(
         flimflam::kTypeProperty,
         base::Value::CreateStringValue(type));
     properties->SetWithoutPathExpansion(
         flimflam::kStateProperty,
         base::Value::CreateStringValue(state));
-    properties->SetWithoutPathExpansion(
-        flimflam::kSecurityProperty,
-        base::Value::CreateStringValue(security));
   }
 
   virtual void RemoveService(const std::string& service_path) {
     stub_services_.RemoveWithoutPathExpansion(service_path, NULL);
   }
 
   virtual void SetServiceProperty(const std::string& service_path,
                                   const std::string& property,
                                   const base::Value& value) OVERRIDE {
     SetProperty(dbus::ObjectPath(service_path), property, value,
-                base::Bind(&base::DoNothing), ErrorCallback());
+                base::Bind(&base::DoNothing),
+                base::Bind(&ShillServiceClientStubImpl::ErrorFunction));
   }
 
   virtual void ClearServices() OVERRIDE {
     stub_services_.Clear();
   }
 
  private:
   void SetDefaultProperties() {
     // Add stub services. Note: names match Manager stub impl.
     AddService("stub_ethernet", "eth0",
                flimflam::kTypeEthernet,
-               flimflam::kStateOnline,
-               flimflam::kSecurityNone);
+               flimflam::kStateOnline);
+
     AddService("stub_wifi1", "wifi1",
                flimflam::kTypeWifi,
-               flimflam::kStateOnline,
-               flimflam::kSecurityNone);
+               flimflam::kStateOnline);
+
     AddService("stub_wifi2", "wifi2_PSK",
                flimflam::kTypeWifi,
-               flimflam::kStateIdle,
-               flimflam::kSecurityPsk);
+               flimflam::kStateIdle);
+    base::StringValue psk_value(flimflam::kSecurityPsk);
+    SetServiceProperty("stub_wifi2",
+                       flimflam::kSecurityProperty,
+                       psk_value);
+
     AddService("stub_cellular1", "cellular1",
                flimflam::kTypeCellular,
-               flimflam::kStateIdle,
-               flimflam::kSecurityNone);
+               flimflam::kStateIdle);
     base::StringValue technology_value(flimflam::kNetworkTechnologyGsm);
     SetServiceProperty("stub_cellular1",
                        flimflam::kNetworkTechnologyProperty,
                        technology_value);
   }
 
   void PassStubDictionaryValue(const dbus::ObjectPath& service_path,
                                const DictionaryValueCallback& callback) {
     base::DictionaryValue* dict = NULL;
     if (!stub_services_.GetDictionaryWithoutPathExpansion(
@@ skipping 27 matching lines @@
   base::DictionaryValue* GetServiceProperties(const std::string& service_path) {
     base::DictionaryValue* properties = NULL;
     if (!stub_services_.GetDictionaryWithoutPathExpansion(
             service_path, &properties)) {
       properties = new base::DictionaryValue;
       stub_services_.Set(service_path, properties);
     }
     return properties;
   }
 
+  static void ErrorFunction(const std::string& error_name,
+                            const std::string& error_message) {
+    LOG(ERROR) << "Shill Error: " << error_name << " : " << error_message;
+  }
 
   base::DictionaryValue stub_services_;
   ObserverList<ShillPropertyChangedObserver> observer_list_;
 
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<ShillServiceClientStubImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(ShillServiceClientStubImpl);
 };
 
 }  // namespace
 
 ShillServiceClient::ShillServiceClient() {}
 
 ShillServiceClient::~ShillServiceClient() {}
 
 // static
 ShillServiceClient* ShillServiceClient::Create(
     DBusClientImplementationType type,
     dbus::Bus* bus) {
   if (type == REAL_DBUS_CLIENT_IMPLEMENTATION)
     return new ShillServiceClientImpl(bus);
   DCHECK_EQ(STUB_DBUS_CLIENT_IMPLEMENTATION, type);
   return new ShillServiceClientStubImpl();
 }
 
 }  // namespace chromeos