commit 8fe2c7eea92541b5282929361a19ad519e0608a9
author Chris Masone <cmasone@chromium.org> Thu Jun 09 15:51:19 2011 -0700
committer Chris Masone <cmasone@chromium.org> Fri Jun 10 14:45:10 2011 -0700
tree d2e7654569941154062bac0a50de492b840df262
parent 92c4390285270320f6145c8295de0abdcc315380

[shill] Begin wiring SetProperty

Create a PropertyStoreInterface that can be implemented by Device,
Service, Manager, etc to handle getting/setting of properties.  The
various DBusAdaptor subclasses can dispatch incoming calls to their
associated implementation object based on the names of the properties
and/or the types of the arguments.

BUG=chromium-os:16343
TEST=unit tests

Change-Id: If7abb0ffa623e59288943b4ed1f49832a92524ea
Reviewed-on: http://gerrit.chromium.org/gerrit/2408
Reviewed-by: Chris Masone <cmasone@chromium.org>
Tested-by: Chris Masone <cmasone@chromium.org>

device_dbus_adaptor.cc

diff --git a/device_dbus_adaptor.cc b/device_dbus_adaptor.cc
index eda5ec2..9beec49 100644
--- a/device_dbus_adaptor.cc
+++ b/device_dbus_adaptor.cc
@@ -9,6 +9,7 @@
 #include <string>
 
 #include "shill/device.h"
+#include "shill/error.h"
 
 using std::map;
 using std::string;
@@ -36,11 +37,11 @@
 }
 
 void DeviceDBusAdaptor::EmitUintChanged(const std::string& name, uint32 value) {
-  PropertyChanged(name, DBusAdaptor::UInt32ToVariant(value));
+  PropertyChanged(name, DBusAdaptor::Uint32ToVariant(value));
 }
 
 void DeviceDBusAdaptor::EmitIntChanged(const std::string& name, int value) {
-  PropertyChanged(name, DBusAdaptor::IntToVariant(value));
+  PropertyChanged(name, DBusAdaptor::Int32ToVariant(value));
 }
 
 void DeviceDBusAdaptor::EmitStringChanged(const std::string& name,
@@ -53,9 +54,10 @@
   return map<string, ::DBus::Variant>();
 }
 
-void DeviceDBusAdaptor::SetProperty(const string& ,
-                                    const ::DBus::Variant& ,
+void DeviceDBusAdaptor::SetProperty(const string& name,
+                                    const ::DBus::Variant& value,
                                     ::DBus::Error &error) {
+  DBusAdaptor::DispatchOnType(device_, name, value, error);
 }
 
 void DeviceDBusAdaptor::ClearProperty(const std::string& ,