commit 3239932e4a0a92fec52d88860081cbbb003eabb0
author mukesh agrawal <quiche@chromium.org> Thu Sep 01 10:53:43 2011 -0700
committer mukesh agrawal <quiche@chromium.org> Thu Sep 08 14:48:23 2011 -0700
tree d275c2cdf425ee940df8393dcc29aa4e83ee40b1
parent f1f490ce9d3aef0309244f63682a651efa37f1d3

shill: implement manager.RequestScan (for WiFi only)

BUG=chromium-os:19831
TEST=unittests, WiFiManager/000_SSID_Length_Limit

note that 000_SSID_Length_Limit does not pass yet,
because we don't do IP configuration yet. but it
does get scan results.

Change-Id: I8993b3c646eda705271d1f7ad7d7341c692ae06b
Reviewed-on: http://gerrit.chromium.org/gerrit/7407
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>

dbus_adaptor.cc

diff --git a/dbus_adaptor.cc b/dbus_adaptor.cc
index d84ee0a..96672d7 100644
--- a/dbus_adaptor.cc
+++ b/dbus_adaptor.cc
@@ -21,6 +21,8 @@
 namespace shill {
 
 // static
+const char DBusAdaptor::kPathArraySig[] = "ao";
+// static
 const char DBusAdaptor::kStringmapSig[] = "a{ss}";
 // static
 const char DBusAdaptor::kStringmapsSig[] = "aa{ss}";
@@ -166,6 +168,19 @@
 }
 
 // static
+::DBus::Variant DBusAdaptor::PathArrayToVariant(
+    const vector< ::DBus::Path> &value) {
+  ::DBus::MessageIter writer;
+  ::DBus::Variant v;
+
+  // TODO(quiche): figure out why we can't use operator<< without the
+  // temporary variable.
+  writer = v.writer();
+  writer << value;
+  return v;
+}
+
+// static
 ::DBus::Variant DBusAdaptor::StringToVariant(const string &value) {
   ::DBus::Variant v;
   v.writer().append_string(value.c_str());
@@ -245,6 +260,11 @@
 }
 
 // static
+bool DBusAdaptor::IsPathArray(::DBus::Signature signature) {
+  return signature == DBusAdaptor::kPathArraySig;
+}
+
+// static
 bool DBusAdaptor::IsString(::DBus::Signature signature) {
   return signature == ::DBus::type<string>::sig();
 }