Add FlimflamIPConfigClient

chrome/browser/chromeos/dbus/flimflam_ipconfig_client.h

Index: chrome/browser/chromeos/dbus/flimflam_ipconfig_client.h
diff --git a/chrome/browser/chromeos/dbus/flimflam_ipconfig_client.h b/chrome/browser/chromeos/dbus/flimflam_ipconfig_client.h
new file mode 100644
index 0000000000000000000000000000000000000000..9e24b79606d9fdc892b64b79fe33fed758bdb500
--- /dev/null
+++ b/chrome/browser/chromeos/dbus/flimflam_ipconfig_client.h
@@ -0,0 +1,94 @@
+// Copyright (c) 2012 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_CHROMEOS_DBUS_FLIMFLAM_IPCONFIG_CLIENT_H_
+#define CHROME_BROWSER_CHROMEOS_DBUS_FLIMFLAM_IPCONFIG_CLIENT_H_
+
+#include <string>
+
+#include "base/basictypes.h"
+#include "base/callback.h"
+
+namespace base {
+
+class Value;
+class DictionaryValue;
+
+}  // namespace base
+
+namespace dbus {
+
+class Bus;
+class ObjectPath;
+
+}  // namespace dbus
+
+namespace chromeos {
+
+// FlimflamIPConfigClient is used to communicate with the Flimflam IPConfig
+// service.  All methods should be called from the origin thread which
+// initializes the DBusThreadManager instance.
+class FlimflamIPConfigClient {
+ public:
+  // An enum to describe whether or not a DBus method call succeeded.
+  enum CallStatus {
+    FAILURE,
+    SUCCESS,
+  };
+
+  // A callback to handle PropertyChanged signals.
+  typedef base::Callback<void(const std::string& name,
+                              const base::Value& value)> PropertyChangedHandler;
+
+  // A callback to handle responses of methods without results.
+  typedef base::Callback<void(CallStatus call_status)> VoidCallback;
+
+  // A callback to handle responses of methods with DictionaryValue results.
+  typedef base::Callback<void(
+      CallStatus call_status,
+      const base::DictionaryValue& result)> DictionaryValueCallback;
+
+  virtual ~FlimflamIPConfigClient();
+
+  // Factory function, creates a new instance and returns ownership.

[stevenjb, 2012/03/29 00:28:14]

nit: 'and returns ownership' is confusing. Prefer 'which is owned by the
caller.'

[hashimoto, 2012/03/29 03:04:46]

Done.

+  // For normal usage, access the singleton via DBusThreadManager::Get().
+  static FlimflamIPConfigClient* Create(dbus::Bus* bus);
+
+  // Sets PropertyChanged signal handler.
+  virtual void SetPropertyChangedHandler(
+      const PropertyChangedHandler& handler) = 0;
+
+  // Resets PropertyChanged signal handler.
+  virtual void ResetPropertyChangedHandler() = 0;
+
+  // Calls GetProperties method.
+  // |callback| is called after the method call succeeds.
+  virtual void GetProperties(const DictionaryValueCallback& callback) = 0;
+
+  // Calls SetProperty method.
+  // |callback| is called after the method call succeeds.
+  virtual void SetProperty(const std::string& name,
+                           const base::Value& value,
+                           const VoidCallback& callback) = 0;
+
+  // Calls ClearProperty method.
+  // |callback| is called after the method call succeeds.
+  virtual void ClearProperty(const std::string& name,
+                             const VoidCallback& callback) = 0;
+
+  // Calls Remove method.
+  // |callback| is called after the method call succeeds.
+  virtual void Remove(const VoidCallback& callback) = 0;
+
+ protected:
+  // Create() should be used instead.
+  FlimflamIPConfigClient();
+
+ private:
+  DISALLOW_COPY_AND_ASSIGN(FlimflamIPConfigClient);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_CHROMEOS_DBUS_FLIMFLAM_IPCONFIG_CLIENT_H_