Enumerate HID devices asynchronously on Windows.

device/hid/hid_service_win.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_HID_HID_SERVICE_WIN_H_
 #define DEVICE_HID_HID_SERVICE_WIN_H_
 
 #include <windows.h>
 #include <hidclass.h>
 
 extern "C" {
 #include <hidsdi.h>
 #include <hidpi.h>
 }
 
 #include "base/memory/scoped_ptr.h"
+#include "base/memory/weak_ptr.h"
 #include "base/scoped_observer.h"
 #include "base/win/scoped_handle.h"
 #include "device/core/device_monitor_win.h"
 #include "device/hid/hid_device_info.h"
 #include "device/hid/hid_service.h"
 
 namespace base {
 namespace win {
 class MessageWindow;
 }
 }
 
 namespace device {
 
 class HidServiceWin : public HidService, public DeviceMonitorWin::Observer {
  public:
-  HidServiceWin();
+  HidServiceWin(scoped_refptr<base::SingleThreadTaskRunner> file_task_runner);
 
   virtual void Connect(const HidDeviceId& device_id,
                        const ConnectCallback& callback) override;
 
  private:
   virtual ~HidServiceWin();
 
-  void DoInitialEnumeration();
+  static void InitialEnumerationFileThread(

[Ken Rockot(use gerrit already), 2015/02/18 20:40:23]

nit: Judging from its ubiquity across the codebase I would suggest that an
idiomatic suffix would be "OnFileThread" for this method (and AddDevice...
below).

On a related note, I'd hesitate to use any non-accessor method name that isn't a
verb phrase. Would this be OK as EnumerateOnFileThread? Or maybe
DoFirstEnumerationOnFileThread if you think the "initial" qualification is
important?

+      base::WeakPtr<HidServiceWin> service,
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner);
   static void CollectInfoFromButtonCaps(PHIDP_PREPARSED_DATA preparsed_data,
                                         HIDP_REPORT_TYPE report_type,
                                         USHORT button_caps_length,
                                         HidCollectionInfo* collection_info);
   static void CollectInfoFromValueCaps(PHIDP_PREPARSED_DATA preparsed_data,
                                        HIDP_REPORT_TYPE report_type,
                                        USHORT value_caps_length,
                                        HidCollectionInfo* collection_info);
+  static void AddDeviceFileThread(
+      base::WeakPtr<HidServiceWin> service,
+      scoped_refptr<base::SingleThreadTaskRunner> task_runner,
+      const std::string& device_path);
 
   // DeviceMonitorWin::Observer implementation:
   void OnDeviceAdded(const std::string& device_path) override;
   void OnDeviceRemoved(const std::string& device_path) override;
 
   // Tries to open the device read-write and falls back to read-only.
-  base::win::ScopedHandle OpenDevice(const std::string& device_path);
+  static base::win::ScopedHandle OpenDevice(const std::string& device_path);
 
   scoped_refptr<base::SingleThreadTaskRunner> task_runner_;
+  scoped_refptr<base::SingleThreadTaskRunner> file_task_runner_;
   ScopedObserver<DeviceMonitorWin, DeviceMonitorWin::Observer> device_observer_;
+  base::WeakPtrFactory<HidServiceWin> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(HidServiceWin);
 };
 
 }  // namespace device
 
 #endif  // DEVICE_HID_HID_SERVICE_WIN_H_