Improve functionality of FakeShillProfileClient

chromeos/dbus/fake_shill_profile_client.h

Index: chromeos/dbus/fake_shill_profile_client.h
diff --git a/chromeos/dbus/fake_shill_profile_client.h b/chromeos/dbus/fake_shill_profile_client.h
index 9353f50f26416387f5bb33cdebf3cd4ec84be9df..94d5750eca6ac77085af5de56faff84fe63a5548 100644
--- a/chromeos/dbus/fake_shill_profile_client.h
+++ b/chromeos/dbus/fake_shill_profile_client.h
@@ -53,12 +53,22 @@ class CHROMEOS_EXPORT FakeShillProfileClient :
                         const base::DictionaryValue& properties) OVERRIDE;
   virtual bool AddService(const std::string& profile_path,
                           const std::string& service_path) OVERRIDE;
+  virtual bool UpdateService(const std::string& profile_path,
+                             const std::string& service_path) OVERRIDE;
   virtual void GetProfilePaths(std::vector<std::string>* profiles) OVERRIDE;
+  virtual bool GetService(const std::string& service_path,
+                          std::string* profile_path,
+                          base::DictionaryValue* properties) OVERRIDE;
+  virtual void ClearProfiles() OVERRIDE;
 
  private:
   struct ProfileProperties;
   typedef std::map<std::string, ProfileProperties*> ProfileMap;
 
+  bool AddOrUpdateServiceImpl(ProfileProperties* profile,

[pneubeck (no reviews), 2014/05/14 08:12:17]

argument order: profile (because non-const) should be at the end.

[stevenjb, 2014/05/14 17:08:10]

Done.

+                              const std::string& profile_path,
+                              const std::string& service_path);
+
   ProfileProperties* GetProfile(const dbus::ObjectPath& profile_path,
                                 const ErrorCallback& error_callback);