Rely on Service.Visible instead of Manager.Services

With this change we rely on Service.Visible and only use
Manager.Services to determine which services to observe.

BUG=384380
R=pneubeck@chromium.org

Review URL: https://codereview.chromium.org/330833003

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278594 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 11 insertions, 2 deletions

diff --git a/chromeos/dbus/shill_service_client.h b/chromeos/dbus/shill_service_client.h
index 40507fe..e8b950f 100644
--- a/chromeos/dbus/shill_service_client.h
+++ b/chromeos/dbus/shill_service_client.h
@@ -48,14 +48,23 @@ class CHROMEOS_EXPORT ShillServiceClient : public DBusClient {
                             const std::string& name,
                             const std::string& type,
                             const std::string& state,
-                            bool add_to_visible_list) = 0;
+                            bool visible) = 0;
     virtual void AddServiceWithIPConfig(const std::string& service_path,
                                         const std::string& guid,
                                         const std::string& name,
                                         const std::string& type,
                                         const std::string& state,
                                         const std::string& ipconfig_path,
-                                        bool add_to_visible_list) = 0;
+                                        bool visible) = 0;
+    // Sets the properties for a service but does not add it to the Manager
+    // or Profile. Returns the properties for the service.
+    virtual base::DictionaryValue* SetServiceProperties(
+        const std::string& service_path,
+        const std::string& guid,
+        const std::string& name,
+        const std::string& type,
+        const std::string& state,
+        bool visible) = 0;
 
     // Removes a Service to the Manager and Service stubs.
     virtual void RemoveService(const std::string& service_path) = 0;