shill: Cleanup some leaks of DBus-specific types into core shill code.

Also, generalize DBusAdaptor::KeyValueStoreToVariant a bit by using a
more complete DBusProperties::ConvertKeyValueStoreToMap method and add
unit tests.

BUG=chrome-os-partner:9837
TEST=unit tests

Change-Id: Ie8396bc2b0e46b2982360b2af5c34950a1b1015d
Reviewed-on: https://gerrit.chromium.org/gerrit/23122
Tested-by: Darin Petkov <petkov@chromium.org>
Reviewed-by: Jason Glasgow <jglasgow@chromium.org>
Reviewed-by: Ben Chan <benchan@chromium.org>
Commit-Ready: Darin Petkov <petkov@chromium.org>
diff --git a/mock_wimax_device_proxy.h b/mock_wimax_device_proxy.h
index 3ff2c0b..2b1412a 100644
--- a/mock_wimax_device_proxy.h
+++ b/mock_wimax_device_proxy.h
@@ -26,8 +26,8 @@
   MOCK_METHOD3(ScanNetworks, void(Error *error,
                                   const ResultCallback &callback,
                                   int timeout));
-  MOCK_METHOD5(Connect, void(const DBus::Path &network,
-                             const DBusPropertiesMap &parameters,
+  MOCK_METHOD5(Connect, void(const RpcIdentifier &network,
+                             const KeyValueStore &parameters,
                              Error *error,
                              const ResultCallback &callback,
                              int timeout));