Change webcamPrivate.set/get/reset to async APIs.

extensions/browser/api/webcam_private/webcam_private_api.h

 // Copyright 2014 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 EXTENSIONS_BROWSER_API_WEBCAM_PRIVATE_WEBCAM_PRIVATE_API_H_
 #define EXTENSIONS_BROWSER_API_WEBCAM_PRIVATE_WEBCAM_PRIVATE_API_H_
 
 #include <map>
 
 #include "base/memory/scoped_ptr.h"
@@ skipping 73 matching lines @@
     ::DeclareFactoryDependencies();
 
 class WebcamPrivateOpenSerialWebcamFunction : public AsyncExtensionFunction {
  public:
   WebcamPrivateOpenSerialWebcamFunction();
   DECLARE_EXTENSION_FUNCTION("webcamPrivate.openSerialWebcam",
                              WEBCAMPRIVATE_OPENSERIALWEBCAM);
 
  protected:
   ~WebcamPrivateOpenSerialWebcamFunction() override;
-
-  // AsyncExtensionFunction:
   bool RunAsync() override;
 
  private:
   void OnOpenWebcam(const std::string& webcam_id, bool success);
 
   DISALLOW_COPY_AND_ASSIGN(WebcamPrivateOpenSerialWebcamFunction);
 };
 
 class WebcamPrivateCloseWebcamFunction : public AsyncExtensionFunction {
  public:
   WebcamPrivateCloseWebcamFunction();
   DECLARE_EXTENSION_FUNCTION("webcamPrivate.closeWebcam",
                              WEBCAMPRIVATE_CLOSEWEBCAM);
 
  protected:
   ~WebcamPrivateCloseWebcamFunction() override;
-
-  // AsyncApiFunction:

[Zachary Kuznia, 2015/08/19 20:49:44]

Why did you remove these comments?

[xdai1, 2015/08/21 22:57:03]

It seems a little redundant? I noticed that several classes in this file don't
have this kind of comments while other do. Besides, if we do want to add "//
AsyncApiFunction" comment here, maybe we should move it before
"~WebcamPrivateCloseWebcamFunction()" since it's also an override function?

   bool RunAsync() override;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(WebcamPrivateCloseWebcamFunction);
 };
 
-class WebcamPrivateSetFunction : public SyncExtensionFunction {
+class WebcamPrivateSetFunction : public AsyncExtensionFunction {
  public:
   WebcamPrivateSetFunction();
   DECLARE_EXTENSION_FUNCTION("webcamPrivate.set", WEBCAMPRIVATE_SET);
 
  protected:
   ~WebcamPrivateSetFunction() override;
-  bool RunSync() override;
+  bool RunAsync() override;
 
  private:
+  void OnSetWebcamParameters(bool success);
+
   DISALLOW_COPY_AND_ASSIGN(WebcamPrivateSetFunction);
 };
 
-class WebcamPrivateGetFunction : public SyncExtensionFunction {
+class WebcamPrivateGetFunction : public AsyncExtensionFunction {
  public:
   WebcamPrivateGetFunction();
   DECLARE_EXTENSION_FUNCTION("webcamPrivate.get", WEBCAMPRIVATE_GET);
 
  protected:
   ~WebcamPrivateGetFunction() override;
-  bool RunSync() override;
+  bool RunAsync() override;
 
  private:
+  void OnGetWebcamParameters(bool success, int value);
+
   DISALLOW_COPY_AND_ASSIGN(WebcamPrivateGetFunction);
 };
 
-class WebcamPrivateResetFunction : public SyncExtensionFunction {
+class WebcamPrivateResetFunction : public AsyncExtensionFunction {
  public:
   WebcamPrivateResetFunction();
   DECLARE_EXTENSION_FUNCTION("webcamPrivate.reset", WEBCAMPRIVATE_RESET);
 
  protected:
   ~WebcamPrivateResetFunction() override;
-  bool RunSync() override;
+  bool RunAsync() override;
 
  private:
+  void OnResetWebcam(bool success);
+
   DISALLOW_COPY_AND_ASSIGN(WebcamPrivateResetFunction);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_WEBCAM_PRIVATE_WEBCAM_PRIVATE_API_H_