Shill: ShillServiceClient destroys ShillClientHelpers when they are not need.

chromeos/dbus/shill_service_client_unittest.cc

Index: chromeos/dbus/shill_service_client_unittest.cc
diff --git a/chromeos/dbus/shill_service_client_unittest.cc b/chromeos/dbus/shill_service_client_unittest.cc
index 736c215a96c1f61f2cb24ac7e8eaa23613f39f81..8ff33902595888da9e0162175f4bdd8b9b1fed49 100644
--- a/chromeos/dbus/shill_service_client_unittest.cc
+++ b/chromeos/dbus/shill_service_client_unittest.cc
@@ -64,6 +64,10 @@ TEST_F(ShillServiceClientTest, PropertyChanged) {
                   flimflam::kSignalStrengthProperty,
                   ValueEq(ByRef(value)))).Times(1);
 
+  // The ShillClientHelper should be created here because of this call.
+  EXPECT_CALL(*mock_bus_, RemoveObjectProxy(_,
+      dbus::ObjectPath(kExampleServicePath), _)).Times(0);
+
   // Add the observer
   client_->AddPropertyChangedObserver(
       dbus::ObjectPath(kExampleServicePath),
@@ -72,6 +76,9 @@ TEST_F(ShillServiceClientTest, PropertyChanged) {
   // Run the signal callback.
   SendPropertyChangedSignal(&signal);
 
+  EXPECT_CALL(*mock_bus_, RemoveObjectProxy(_,
+      dbus::ObjectPath(kExampleServicePath), _)).Times(1);
+
   // Remove the observer.
   client_->RemovePropertyChangedObserver(
       dbus::ObjectPath(kExampleServicePath),