Add NetworkingPrivateDelegate class

chrome/browser/extensions/api/networking_private/networking_private_api.h

 // Copyright 2013 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.
 
 // These classes implement the chrome.networkingPrivate JavaScript extension
 // API.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_NETWORKING_PRIVATE_NETWORKING_PRIVATE_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_NETWORKING_PRIVATE_NETWORKING_PRIVATE_API_H_
 
 #include <string>
 
 #include "base/memory/ref_counted.h"

[pneubeck (no reviews), 2014/07/14 21:33:57]

unused?

missing:
#include "base/macros.h"
#include "base/memory/scoped_ptr.h"

[stevenjb, 2014/07/15 00:33:39]

Done.

 #include "base/values.h"
-#include "chrome/browser/extensions/chrome_extension_function.h"
 #include "chromeos/dbus/dbus_method_call_status.h"

[pneubeck (no reviews), 2014/07/14 21:33:57]

unused?

[stevenjb, 2014/07/15 00:33:39]

Done.

 #include "components/keyed_service/core/keyed_service.h"

[pneubeck (no reviews), 2014/07/14 21:33:57]

unused?

[stevenjb, 2014/07/15 00:33:39]

Done.

+#include "extensions/browser/extension_function.h"
+
+namespace extensions {
+
+namespace networking_private {
+
+extern const char kErrorInvalidNetworkGuid[];
+extern const char kErrorNetworkUnavailable[];
+extern const char kErrorNotReady[];
+extern const char kErrorNotSupported[];
+
+}  // namespace networking_private
 
 // Implements the chrome.networkingPrivate.getProperties method.
-class NetworkingPrivateGetPropertiesFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateGetPropertiesFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetPropertiesFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getProperties",
                              NETWORKINGPRIVATE_GETPROPERTIES);
 
  protected:
   virtual ~NetworkingPrivateGetPropertiesFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void GetPropertiesSuccess(const std::string& service_path,
-                            const base::DictionaryValue& result);
-  void GetPropertiesFailed(const std::string& error_name,
-                           scoped_ptr<base::DictionaryValue> error_data);
+  void Success(scoped_ptr<base::DictionaryValue> result);
+  void Failure(const std::string& error_name);
+
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetPropertiesFunction);
 };
 
 // Implements the chrome.networkingPrivate.getManagedProperties method.
 class NetworkingPrivateGetManagedPropertiesFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetManagedPropertiesFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getManagedProperties",
                              NETWORKINGPRIVATE_GETMANAGEDPROPERTIES);
 
  protected:
   virtual ~NetworkingPrivateGetManagedPropertiesFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  // Callbacks for ManagedNetworkConfigurationHandler::GetManagedProperties.
-  void Success(const std::string& service_path,
-               const base::DictionaryValue& result);
-  void Failure(const std::string& error_name,
-              scoped_ptr<base::DictionaryValue> error_data);
+  void Success(scoped_ptr<base::DictionaryValue> result);
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetManagedPropertiesFunction);
 };
 
 // Implements the chrome.networkingPrivate.getState method.
-class NetworkingPrivateGetStateFunction : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateGetStateFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetStateFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getState",
                              NETWORKINGPRIVATE_GETSTATE);
 
  protected:
   virtual ~NetworkingPrivateGetStateFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void Success(const std::string& service_path,
-               const base::DictionaryValue& result);
-  void Failure(const std::string& error_name,
-               scoped_ptr<base::DictionaryValue> error_data);
+  void Success(scoped_ptr<base::DictionaryValue> result);
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetStateFunction);
 };
 
 // Implements the chrome.networkingPrivate.setProperties method.
-class NetworkingPrivateSetPropertiesFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateSetPropertiesFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateSetPropertiesFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.setProperties",
                              NETWORKINGPRIVATE_SETPROPERTIES);
 
  protected:
   virtual ~NetworkingPrivateSetPropertiesFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void ErrorCallback(const std::string& error_name,
-                     const scoped_ptr<base::DictionaryValue> error_data);
-  void ResultCallback();
+  void Success();
+  void Failure(const std::string& error);
+
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateSetPropertiesFunction);
 };
 
 // Implements the chrome.networkingPrivate.createNetwork method.
-class NetworkingPrivateCreateNetworkFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateCreateNetworkFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateCreateNetworkFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.createNetwork",
                              NETWORKINGPRIVATE_CREATENETWORK);
 
  protected:
   virtual ~NetworkingPrivateCreateNetworkFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void ErrorCallback(const std::string& error_name,
-                     const scoped_ptr<base::DictionaryValue> error_data);
-  void ResultCallback(const std::string& guid);
+  void Success(const std::string& guid);
+  void Failure(const std::string& error);
+
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateCreateNetworkFunction);
 };
 
 // Implements the chrome.networkingPrivate.getNetworks method.
-class NetworkingPrivateGetNetworksFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateGetNetworksFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetNetworksFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getNetworks",
                              NETWORKINGPRIVATE_GETNETWORKS);
 
  protected:
   virtual ~NetworkingPrivateGetNetworksFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void ResultCallback(const base::ListValue& network_list);
+  void Success(scoped_ptr<base::ListValue> network_list);
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetNetworksFunction);
 };
 
 // Implements the chrome.networkingPrivate.getVisibleNetworks method.
 class NetworkingPrivateGetVisibleNetworksFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetVisibleNetworksFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getVisibleNetworks",
                              NETWORKINGPRIVATE_GETVISIBLENETWORKS);
 
  protected:
   virtual ~NetworkingPrivateGetVisibleNetworksFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  void ResultCallback(const base::ListValue& network_list);
+  void Success(scoped_ptr<base::ListValue> network_list);
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetVisibleNetworksFunction);
 };
 
 // Implements the chrome.networkingPrivate.getEnabledNetworkTypes method.
 class NetworkingPrivateGetEnabledNetworkTypesFunction
-    : public ChromeSyncExtensionFunction {
+    : public SyncExtensionFunction {
  public:
   NetworkingPrivateGetEnabledNetworkTypesFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getEnabledNetworkTypes",
                              NETWORKINGPRIVATE_GETENABLEDNETWORKTYPES);
 
  protected:
   virtual ~NetworkingPrivateGetEnabledNetworkTypesFunction();
 
   // SyncExtensionFunction overrides.
   virtual bool RunSync() OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetEnabledNetworkTypesFunction);
 };
 
 // Implements the chrome.networkingPrivate.enableNetworkType method.
 class NetworkingPrivateEnableNetworkTypeFunction
-    : public ChromeSyncExtensionFunction {
+    : public SyncExtensionFunction {
  public:
   NetworkingPrivateEnableNetworkTypeFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.enableNetworkType",
                              NETWORKINGPRIVATE_ENABLENETWORKTYPE);
 
  protected:
   virtual ~NetworkingPrivateEnableNetworkTypeFunction();
 
   // SyncExtensionFunction overrides.
   virtual bool RunSync() OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateEnableNetworkTypeFunction);
 };
 
 // Implements the chrome.networkingPrivate.disableNetworkType method.
 class NetworkingPrivateDisableNetworkTypeFunction
-    : public ChromeSyncExtensionFunction {
+    : public SyncExtensionFunction {
  public:
   NetworkingPrivateDisableNetworkTypeFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.disableNetworkType",
                              NETWORKINGPRIVATE_DISABLENETWORKTYPE);
 
  protected:
   virtual ~NetworkingPrivateDisableNetworkTypeFunction();
 
   // SyncExtensionFunction overrides.
   virtual bool RunSync() OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateDisableNetworkTypeFunction);
 };
 
 // Implements the chrome.networkingPrivate.requestNetworkScan method.
 class NetworkingPrivateRequestNetworkScanFunction
-    : public ChromeSyncExtensionFunction {
+    : public SyncExtensionFunction {
  public:
   NetworkingPrivateRequestNetworkScanFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.requestNetworkScan",
                              NETWORKINGPRIVATE_REQUESTNETWORKSCAN);
 
  protected:
   virtual ~NetworkingPrivateRequestNetworkScanFunction();
 
   // SyncExtensionFunction overrides.
   virtual bool RunSync() OVERRIDE;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateRequestNetworkScanFunction);
 };
 
 
 // Implements the chrome.networkingPrivate.startConnect method.
-class NetworkingPrivateStartConnectFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateStartConnectFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateStartConnectFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.startConnect",
                              NETWORKINGPRIVATE_STARTCONNECT);
 
  protected:
   virtual ~NetworkingPrivateStartConnectFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  // Called when the request to connect succeeds. Doesn't mean that the connect
-  // itself succeeded, just that the request did.
-  void ConnectionStartSuccess();
-
-  void ConnectionStartFailed(
-      const std::string& error_name,
-      const scoped_ptr<base::DictionaryValue> error_data);
+  void Success();
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateStartConnectFunction);
 };
 
 // Implements the chrome.networkingPrivate.startDisconnect method.
-class NetworkingPrivateStartDisconnectFunction
-    : public ChromeAsyncExtensionFunction {
+class NetworkingPrivateStartDisconnectFunction : public AsyncExtensionFunction {
  public:
   NetworkingPrivateStartDisconnectFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.startDisconnect",
                              NETWORKINGPRIVATE_STARTDISCONNECT);
 
  protected:
   virtual ~NetworkingPrivateStartDisconnectFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  private:
-  // Called when the request to disconnect succeeds. Doesn't mean that the
-  // disconnect itself succeeded, just that the request did.
-  void DisconnectionStartSuccess();
-
-  void DisconnectionStartFailed(
-      const std::string& error_name,
-      const scoped_ptr<base::DictionaryValue> error_data);
+  void Success();
+  void Failure(const std::string& error);
 
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateStartDisconnectFunction);
 };
 
 // Implements the chrome.networkingPrivate.verifyDestination method.
 class NetworkingPrivateVerifyDestinationFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateVerifyDestinationFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.verifyDestination",
                              NETWORKINGPRIVATE_VERIFYDESTINATION);
 
  protected:
   virtual ~NetworkingPrivateVerifyDestinationFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
-  void ResultCallback(bool result);
-  void ErrorCallback(const std::string& error_name, const std::string& error);
+  void Success(bool result);
+  void Failure(const std::string& error);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateVerifyDestinationFunction);
 };
 
 // Implements the chrome.networkingPrivate.verifyAndEncryptCredentials method.
 class NetworkingPrivateVerifyAndEncryptCredentialsFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateVerifyAndEncryptCredentialsFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.verifyAndEncryptCredentials",
                              NETWORKINGPRIVATE_VERIFYANDENCRYPTCREDENTIALS);
 
  protected:
   virtual ~NetworkingPrivateVerifyAndEncryptCredentialsFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
-  void ResultCallback(const std::string& result);
-  void ErrorCallback(const std::string& error_name, const std::string& error);
+  void Success(const std::string& result);
+  void Failure(const std::string& error);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(
       NetworkingPrivateVerifyAndEncryptCredentialsFunction);
 };
 
 // Implements the chrome.networkingPrivate.verifyAndEncryptData method.
 class NetworkingPrivateVerifyAndEncryptDataFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateVerifyAndEncryptDataFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.verifyAndEncryptData",
                              NETWORKINGPRIVATE_VERIFYANDENCRYPTDATA);
 
  protected:
   virtual ~NetworkingPrivateVerifyAndEncryptDataFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
-  void ResultCallback(const std::string& result);
-  void ErrorCallback(const std::string& error_name, const std::string& error);
+  void Success(const std::string& result);
+  void Failure(const std::string& error);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateVerifyAndEncryptDataFunction);
 };
 
 // Implements the chrome.networkingPrivate.setWifiTDLSEnabledState method.
 class NetworkingPrivateSetWifiTDLSEnabledStateFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateSetWifiTDLSEnabledStateFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.setWifiTDLSEnabledState",
                              NETWORKINGPRIVATE_SETWIFITDLSENABLEDSTATE);
 
  protected:
   virtual ~NetworkingPrivateSetWifiTDLSEnabledStateFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
   void Success(const std::string& result);
-  void Failure(const std::string& error_name,
-               scoped_ptr<base::DictionaryValue> error_data);
+  void Failure(const std::string& error);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateSetWifiTDLSEnabledStateFunction);
 };
 
 // Implements the chrome.networkingPrivate.getWifiTDLSStatus method.
 class NetworkingPrivateGetWifiTDLSStatusFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetWifiTDLSStatusFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getWifiTDLSStatus",
                              NETWORKINGPRIVATE_GETWIFITDLSSTATUS);
 
  protected:
   virtual ~NetworkingPrivateGetWifiTDLSStatusFunction();
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
   void Success(const std::string& result);
-  void Failure(const std::string& error_name,
-               scoped_ptr<base::DictionaryValue> error_data);
+  void Failure(const std::string& error);
 
  private:
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetWifiTDLSStatusFunction);
 };
 
 class NetworkingPrivateGetCaptivePortalStatusFunction
-    : public ChromeAsyncExtensionFunction {
+    : public AsyncExtensionFunction {
  public:
   NetworkingPrivateGetCaptivePortalStatusFunction() {}
   DECLARE_EXTENSION_FUNCTION("networkingPrivate.getCaptivePortalStatus",
                              NETWORKINGPRIVATE_GETCAPTIVEPORTALSTATUS);
 
   // AsyncExtensionFunction overrides.
   virtual bool RunAsync() OVERRIDE;
 
  protected:
   virtual ~NetworkingPrivateGetCaptivePortalStatusFunction();
 
  private:
+  void Success(const std::string& result);
+  void Failure(const std::string& error);
+
   DISALLOW_COPY_AND_ASSIGN(NetworkingPrivateGetCaptivePortalStatusFunction);
 };
 
+}  // namespace extensions
+
 #endif  // CHROME_BROWSER_EXTENSIONS_API_NETWORKING_PRIVATE_NETWORKING_PRIVATE_API_H_