Get all the UsbConfigDescriptor for the device configuration

device/usb/usb_device_impl.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 DEVICE_USB_USB_DEVICE_IMPL_H_
 #define DEVICE_USB_USB_DEVICE_IMPL_H_
 
-#include <vector>

[Reilly Grant (use Gerrit), 2015/08/03 18:04:04]

Don't remove these includes just because usb_device.h includes vector and
usb_descriptors.h.

See:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Names_and_Ord...

[juncai, 2015/08/03 21:56:25]

Done.

-
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/threading/thread_checker.h"
-#include "device/usb/usb_descriptors.h"
 #include "device/usb/usb_device.h"
 
 struct libusb_device;
 struct libusb_config_descriptor;
 struct libusb_device_handle;
 
 namespace base {
 class SequencedTaskRunner;
 }
 
@@ skipping 11 matching lines @@
 typedef struct libusb_device_handle* PlatformUsbDeviceHandle;
 
 class UsbDeviceImpl : public UsbDevice {
  public:
 // UsbDevice implementation:
 #if defined(OS_CHROMEOS)
   void CheckUsbAccess(const ResultCallback& callback) override;
 #endif  // OS_CHROMEOS
   void Open(const OpenCallback& callback) override;
   bool Close(scoped_refptr<UsbDeviceHandle> handle) override;
-  const UsbConfigDescriptor* GetConfiguration() override;
+  const UsbConfigDescriptor* GetActiveConfiguration() override;
 
   // These functions are used during enumeration only. The values must not
   // change during the object's lifetime.
   void set_manufacturer_string(const base::string16& value) {
     manufacturer_string_ = value;
   }
   void set_product_string(const base::string16& value) {
     product_string_ = value;
   }
   void set_serial_number(const base::string16& value) {
@@ skipping 13 matching lines @@
                 uint16 vendor_id,
                 uint16 product_id,
                 scoped_refptr<base::SequencedTaskRunner> blocking_task_runner);
 
   ~UsbDeviceImpl() override;
 
   // Called only by UsbServiceImpl.
   void set_visited(bool visited) { visited_ = visited; }
   bool was_visited() const { return visited_; }
   void OnDisconnect();
+  void set_configurations();
 
   // Called by UsbDeviceHandleImpl.
   void RefreshConfiguration();
 
  private:
+  void GetAllConfigurations();
 #if defined(OS_CHROMEOS)
   void OnOpenRequestComplete(const OpenCallback& callback,
                              dbus::FileDescriptor fd);
   void OpenOnBlockingThreadWithFd(dbus::FileDescriptor fd,
                                   const OpenCallback& callback);
 #endif
   void OpenOnBlockingThread(const OpenCallback& callback);
   void Opened(PlatformUsbDeviceHandle platform_handle,
               const OpenCallback& callback);
 
   base::ThreadChecker thread_checker_;
   PlatformUsbDevice platform_device_;
   bool visited_ = false;
 
   // On Chrome OS device path is necessary to request access from the permission
   // broker.
   std::string device_path_;
 
   // The current device configuration descriptor. May be null if the device is
   // in an unconfigured state.
-  scoped_ptr<UsbConfigDescriptor> configuration_;
+  const UsbConfigDescriptor* configuration_;

[Reilly Grant (use Gerrit), 2015/08/03 18:04:04]

Maybe call this "active_configuration_" and document that it is a pointer into
configurations_.

[juncai, 2015/08/03 21:56:25]

Done.

 
   // Retain the context so that it will not be released before UsbDevice.
   scoped_refptr<UsbContext> context_;
 
   // Opened handles.
   typedef std::vector<scoped_refptr<UsbDeviceHandleImpl> > HandlesVector;
   HandlesVector handles_;
 
   scoped_refptr<base::SequencedTaskRunner> task_runner_;
   scoped_refptr<base::SequencedTaskRunner> blocking_task_runner_;
 
   DISALLOW_COPY_AND_ASSIGN(UsbDeviceImpl);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_USB_USB_DEVICE_IMPL_H_