commit 6bb9e7c2c068fc15333babea8e66f02332683e33
author mukesh agrawal <quiche@chromium.org> Mon Jan 30 14:57:54 2012 -0800
committer Gerrit <chrome-bot@google.com> Wed Feb 08 20:14:42 2012 -0800
tree 6794af2dc82d91396e429d9efac77340d139a398
parent 8abd2f67616c3d138346781a49a40d4ae2bc7019

shill: rename DispatchOnType to SetProperty

I think this name is clearer. While the function does
have to dispatch based on type, what the caller is trying
to do is to set a property.

BUG=None
TEST=unit tests
Change-Id: I7eedd5a74de7f465310d07271e61cff320645fd8
Reviewed-on: https://gerrit.chromium.org/gerrit/15538
Reviewed-by: mukesh agrawal <quiche@chromium.org>
Tested-by: mukesh agrawal <quiche@chromium.org>
Commit-Ready: mukesh agrawal <quiche@chromium.org>

property_store_unittest.cc

diff --git a/property_store_unittest.cc b/property_store_unittest.cc
index ec1ee9b..33a77f6 100644
--- a/property_store_unittest.cc
+++ b/property_store_unittest.cc
@@ -86,7 +86,7 @@
   // Ensure that an attempt to write unknown properties returns InvalidProperty.
   PropertyStore store;
   ::DBus::Error error;
-  EXPECT_FALSE(DBusAdaptor::DispatchOnType(&store, "", GetParam(), &error));
+  EXPECT_FALSE(DBusAdaptor::SetProperty(&store, "", GetParam(), &error));
   EXPECT_EQ(invalid_prop(), error.name());
 }
 
@@ -214,7 +214,7 @@
 TEST_F(PropertyStoreTest, SetStringmapsProperty) {
   PropertyStore store;
   ::DBus::Error error;
-  EXPECT_FALSE(DBusAdaptor::DispatchOnType(
+  EXPECT_FALSE(DBusAdaptor::SetProperty(
       &store, "", PropertyStoreTest::kStringmapsV, &error));
   EXPECT_EQ(internal_error(), error.name());
 }