Introducing chrome.usb.getDevices/openDevice API

chrome/browser/extensions/api/usb/usb_api.h

 // 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_EXTENSIONS_API_USB_USB_API_H_
 #define CHROME_BROWSER_EXTENSIONS_API_USB_USB_API_H_
 
 #include <string>
 #include <vector>
 
@@ skipping 46 matching lines @@
                    scoped_refptr<net::IOBuffer> data,
                    size_t length);
 };
 
 class UsbFindDevicesFunction : public UsbAsyncApiFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("usb.findDevices", USB_FINDDEVICES)
 
   UsbFindDevicesFunction();
 
-  static void SetDeviceForTest(UsbDevice* device);
-
  protected:
   virtual ~UsbFindDevicesFunction();
 
   virtual bool Prepare() OVERRIDE;
   virtual void AsyncWorkStart() OVERRIDE;
 
  private:
   void EnumerationCompletedFileThread(
       scoped_ptr<std::vector<scoped_refptr<UsbDevice> > > devices);
 
   // This should be run on the IO thread.
   // Create ApiResources and reply.
   void OnCompleted();
 
   scoped_ptr<base::ListValue> result_;
   std::vector<scoped_refptr<UsbDeviceHandle> > device_handles_;
   scoped_ptr<extensions::api::usb::FindDevices::Params> parameters_;
 };
 
+class UsbGetDevicesFunction : public UsbAsyncApiFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION("usb.getDevices", USB_GETDEVICES)
+
+  UsbGetDevicesFunction();
+
+  static void SetDeviceForTest(UsbDevice* device);
+
+  virtual bool Prepare() OVERRIDE;
+  virtual void AsyncWorkStart() OVERRIDE;
+
+ protected:
+  virtual ~UsbGetDevicesFunction();
+
+ private:
+  void EnumerationCompletedFileThread(
+      scoped_ptr<std::vector<scoped_refptr<UsbDevice> > > devices);
+
+  scoped_ptr<base::ListValue> result_;
+  scoped_ptr<extensions::api::usb::GetDevices::Params> parameters_;
+};
+
+class UsbOpenDeviceFunction : public UsbAsyncApiFunction {
+ public:
+  DECLARE_EXTENSION_FUNCTION("usb.openDevice", USB_OPENDEVICE)
+
+  UsbOpenDeviceFunction();
+
+  virtual bool Prepare() OVERRIDE;
+  virtual void AsyncWorkStart() OVERRIDE;
+
+ protected:
+  virtual ~UsbOpenDeviceFunction();
+
+ private:
+  void OpenDeviceOnFile();
+
+  // This should be run on the IO thread.
+  // Create ApiResources and reply.

[asargent_no_longer_on_chrome, 2013/08/22 21:06:13]

Does it actually create a UsbDeviceResource or some other type of ApiResource?

Maybe change this to something like:

// Creates a UsbDeviceResource and sends a reply. Should be run on the 
// IO thread.

[Bei Zhang, 2013/08/23 22:52:00]

This is not needed since we simplified the threading.


On 2013/08/22 21:06:13, Antony Sargent wrote:

+  void OnCompleted();
+
+  scoped_refptr<UsbDeviceHandle> handle_;
+  scoped_ptr<extensions::api::usb::OpenDevice::Params> parameters_;
+};
+
 class UsbListInterfacesFunction : public UsbAsyncApiFunction {
  public:
   DECLARE_EXTENSION_FUNCTION("usb.listInterfaces", USB_LISTINTERFACES)
 
   UsbListInterfacesFunction();
 
  protected:
   virtual ~UsbListInterfacesFunction();
 
   virtual bool Prepare() OVERRIDE;
@@ skipping 173 matching lines @@
 
   // This should be run on the IO thread.
   void OnCompleted(bool success);
   void OnError();
 
   scoped_ptr<extensions::api::usb::ResetDevice::Params> parameters_;
 };
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_USB_USB_API_H_