shill: cellular: simplify property change notification
Simplify property change notification by using a single mechanism to
handle it instead of two. Old style notifications are turned into new
style notifications by the modem object.
A few minor style clean ups -- remove std:: where not needed and
ensure that public interfaces are declared as public interfaces in
subclasses (i.e not protected).
BUG=chromium-os:29569
TEST=run unit tests
Change-Id: If8b957d8b6518c95fa1d20bdef7d1ec6a8ed5ea3
Reviewed-on: https://gerrit.chromium.org/gerrit/20395
Tested-by: Jason Glasgow <jglasgow@chromium.org>
Reviewed-by: Nathan J. Williams <njw@chromium.org>
Commit-Ready: Jason Glasgow <jglasgow@chromium.org>
diff --git a/mock_cellular.h b/mock_cellular.h
index a4c319a..74fd4d7 100644
--- a/mock_cellular.h
+++ b/mock_cellular.h
@@ -29,8 +29,9 @@
virtual ~MockCellular();
MOCK_METHOD2(InitCapability, void(Type, ProxyFactory *));
- MOCK_METHOD1(OnModemManagerPropertiesChanged,
- void(const DBusPropertiesMap &));
+ MOCK_METHOD3(OnDBusPropertiesChanged, void(const std::string &,
+ const DBusPropertiesMap &,
+ const std::vector<std::string> &));
MOCK_METHOD1(set_modem_state, void(ModemState));
private: