This converts the Shill clients to allow propagation of shill errors

chromeos/dbus/mock_shill_service_client.h

Index: chromeos/dbus/mock_shill_service_client.h
diff --git a/chromeos/dbus/mock_shill_service_client.h b/chromeos/dbus/mock_shill_service_client.h
index d8dfb0b97a1a2d6143ef549a293bd8d746889882..12d14bbe78ef55e48c9d43891b2352f1497e10c5 100644
--- a/chromeos/dbus/mock_shill_service_client.h
+++ b/chromeos/dbus/mock_shill_service_client.h
@@ -26,24 +26,29 @@ class MockShillServiceClient : public ShillServiceClient {
                     ShillPropertyChangedObserver* observer));
   MOCK_METHOD2(GetProperties, void(const dbus::ObjectPath& service_path,
                                    const DictionaryValueCallback& callback));
-  MOCK_METHOD4(SetProperty, void(const dbus::ObjectPath& service_path,
+  MOCK_METHOD5(SetProperty, void(const dbus::ObjectPath& service_path,
                                  const std::string& name,
                                  const base::Value& value,
-                                 const VoidDBusMethodCallback& callback));
-  MOCK_METHOD3(ClearProperty, void(const dbus::ObjectPath& service_path,
+                                 const base::Closure& callback,
+                                 const ErrorCallback& error_callback));
+  MOCK_METHOD4(ClearProperty, void(const dbus::ObjectPath& service_path,
                                    const std::string& name,
-                                   const VoidDBusMethodCallback& callback));
+                                   const base::Closure& callback,
+                                   const ErrorCallback& error_callback));
   MOCK_METHOD3(Connect, void(const dbus::ObjectPath& service_path,
                              const base::Closure& callback,
                              const ErrorCallback& error_callback));
-  MOCK_METHOD2(Disconnect, void(const dbus::ObjectPath& service_path,
-                                const VoidDBusMethodCallback& callback));
-  MOCK_METHOD2(Remove, void(const dbus::ObjectPath& service_path,
-                            const VoidDBusMethodCallback& callback));
-  MOCK_METHOD3(ActivateCellularModem,
+  MOCK_METHOD3(Disconnect, void(const dbus::ObjectPath& service_path,
+                                const base::Closure& callback,
+                                const ErrorCallback& error_callback));
+  MOCK_METHOD3(Remove, void(const dbus::ObjectPath& service_path,
+                            const base::Closure& callback,
+                            const ErrorCallback& error_callback));
+  MOCK_METHOD4(ActivateCellularModem,
                void(const dbus::ObjectPath& service_path,
                     const std::string& carrier,
-                    const VoidDBusMethodCallback& callback));
+                    const base::Closure& callback,
+                    const ErrorCallback& error_callback));
   MOCK_METHOD2(CallActivateCellularModemAndBlock,
                bool(const dbus::ObjectPath& service_path,
                     const std::string& carrier));