HID : read-only device access fallback when read/write fails

device/hid/hid_connection_win.cc

 // Copyright (c) 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.
 
 #include "device/hid/hid_connection_win.h"
 
 #include <cstring>
 

[Ken Rockot(use gerrit already), 2014/04/23 23:24:28]

Looks like you forgot to include base/files/file.h here.

[jracle (use Gerrit), 2014/04/24 08:30:48]

Oops wtf

 #include "base/message_loop/message_loop.h"
 #include "base/stl_util.h"
 #include "base/threading/thread_restrictions.h"
 #include "base/win/object_watcher.h"
 #include "base/win/scoped_handle.h"
 #include "device/hid/hid_service.h"
 #include "device/hid/hid_service_win.h"
 
 #define INITGUID
 
@@ skipping 84 matching lines @@
 HidConnectionWin::HidConnectionWin(const HidDeviceInfo& device_info)
     : HidConnection(device_info) {
   DCHECK(thread_checker_.CalledOnValidThread());
   file_.Set(CreateFileA(device_info.device_id.c_str(),
                         GENERIC_WRITE | GENERIC_READ,
                         FILE_SHARE_READ | FILE_SHARE_WRITE,
                         NULL,
                         OPEN_EXISTING,
                         FILE_FLAG_OVERLAPPED,
                         NULL));
+
+  if (!file_.IsValid() &&
+      file_.error_details() == base::File::FILE_ERROR_ACCESS_DENIED) {
+    file_.Set(CreateFileA(device_info.device_id.c_str(),
+                          GENERIC_READ,
+                          FILE_SHARE_READ,
+                          NULL,
+                          OPEN_EXISTING,
+                          FILE_FLAG_OVERLAPPED,
+                          NULL));
+  }
 }
 
 bool HidConnectionWin::available() const {
   return file_.IsValid();
 }
 
 HidConnectionWin::~HidConnectionWin() {
   DCHECK(thread_checker_.CalledOnValidThread());
   CancelIo(file_.Get());
 }
@@ skipping 156 matching lines @@
   }
 }
 
 void HidConnectionWin::OnTransferCanceled(
     scoped_refptr<PendingHidTransfer> transfer) {
   transfers_.erase(transfer);
   transfer->callback_.Run(false, 0);
 }
 
 }  // namespace device