Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(88)

Unified Diff: chromeos/dbus/shill_profile_client.h

Issue 284673004: Improve functionality of FakeShillProfileClient (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « chromeos/dbus/fake_shill_service_client.cc ('k') | chromeos/dbus/shill_service_client.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chromeos/dbus/shill_profile_client.h
diff --git a/chromeos/dbus/shill_profile_client.h b/chromeos/dbus/shill_profile_client.h
index b237fb0524f99767e1c2a1d2e4630f321aec8d9a..9c0f9259c1cc3b953c1a7233823c8eebd001e775 100644
--- a/chromeos/dbus/shill_profile_client.h
+++ b/chromeos/dbus/shill_profile_client.h
@@ -58,14 +58,31 @@ class CHROMEOS_EXPORT ShillProfileClient : public DBusClient {
const base::DictionaryValue& properties) = 0;
// Adds a service to the profile, copying properties from the
- // ShillServiceClient entry (which must be present). Also sets the Profile
- // property of the service in ShillServiceClient.
+ // ShillServiceClient entry matching |service_path|. Returns false if no
+ // Service entry exists or if a Profile entry already exists. Also sets
+ // the Profile property of the service in ShillServiceClient.
virtual bool AddService(const std::string& profile_path,
const std::string& service_path) = 0;
+ // Copies properties from the ShillServiceClient entry matching
+ // |service_path| to the profile entry matching |profile_path|. Returns
+ // false if no Service entry exits or if no Profile entry exists.
+ virtual bool UpdateService(const std::string& profile_path,
+ const std::string& service_path) = 0;
+
// Sets |profiles| to the current list of profile paths.
virtual void GetProfilePaths(std::vector<std::string>* profiles) = 0;
+ // Sets |properties| to the entry for |service_path|, sets |profile_path|
+ // to the path of the profile with the entry, and returns true if the
+ // service exists in any profile.
+ virtual bool GetService(const std::string& service_path,
+ std::string* profile_path,
+ base::DictionaryValue* properties) = 0;
+
+ // Remove all profile entries.
+ virtual void ClearProfiles() = 0;
+
protected:
virtual ~TestInterface() {}
};
« no previous file with comments | « chromeos/dbus/fake_shill_service_client.cc ('k') | chromeos/dbus/shill_service_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698