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

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

- for Windows and Linux
- open device with read access when read/write fails

BUG=364438
R=rockot@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=266022

[jracle (use Gerrit), 2014-04-17 12:57:19]

The CQ bit was checked by jracle@logitech.com

[jracle (use Gerrit), 2014-04-17 12:57:19]

The CQ bit was unchecked by jracle@logitech.com

[jracle (use Gerrit), 2014-04-17 12:59:16]

The CQ bit was checked by jracle@logitech.com

[commit-bot: I haz the power, 2014-04-17 12:59:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jracle@logitech.com/240263003/1

[commit-bot: I haz the power, 2014-04-17 12:59:23]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-17 12:59:23]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[commit-bot: I haz the power, 2014-04-17 12:59:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-17 12:59:34]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[jracle (use Gerrit), 2014-04-17 13:51:10]

Hi Ken,

  I've committed 2 patches (take #2) which aim to open access to read-only
devices, e.g. consumer-control top-level collection. It is described in my
document as improvement #1.

Best,
Julien

[Ken Rockot(use gerrit already), 2014-04-21 20:33:25]

LGTM

https://codereview.chromium.org/240263003/diff/20001/device/hid/hid_connectio...
File device/hid/hid_connection_win.cc (right):

https://codereview.chromium.org/240263003/diff/20001/device/hid/hid_connectio...
device/hid/hid_connection_win.cc:105:
DCHECK(thread_checker_.CalledOnValidThread());
nit: Remove extra vertical whitespace.

[Ken Rockot(use gerrit already), 2014-04-22 23:24:41]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2014-04-22 23:24:46]

The CQ bit was unchecked by rockot@chromium.org

[jracle (use Gerrit), 2014-04-23 07:38:08]

https://codereview.chromium.org/240263003/diff/20001/device/hid/hid_connectio...
File device/hid/hid_connection_win.cc (right):

https://codereview.chromium.org/240263003/diff/20001/device/hid/hid_connectio...
device/hid/hid_connection_win.cc:105:
DCHECK(thread_checker_.CalledOnValidThread());
sure, sorry. Thought DCHECK(...) was the only line impacted (cause it is after
an opening '{')

[Ken Rockot(use gerrit already), 2014-04-23 20:27:24]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-04-23 20:27:45]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jracle@logitech.com/240263003/40001

[commit-bot: I haz the power, 2014-04-23 21:37:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-23 21:37:06]

Try jobs failed on following builders:
  tryserver.chromium on win_chromium_rel

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

https://codereview.chromium.org/240263003/diff/40001/device/hid/hid_connectio...
File device/hid/hid_connection_win.cc (right):

https://codereview.chromium.org/240263003/diff/40001/device/hid/hid_connectio...
device/hid/hid_connection_win.cc:8: 
Looks like you forgot to include base/files/file.h here.

[jracle (use Gerrit), 2014-04-24 08:30:47]

https://codereview.chromium.org/240263003/diff/40001/device/hid/hid_connectio...
File device/hid/hid_connection_win.cc (right):

https://codereview.chromium.org/240263003/diff/40001/device/hid/hid_connectio...
device/hid/hid_connection_win.cc:8: 
Oops wtf

[jracle (use Gerrit), 2014-04-24 08:31:04]

The CQ bit was checked by jracle@logitech.com

[commit-bot: I haz the power, 2014-04-24 08:31:14]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jracle@logitech.com/240263003/60001

[commit-bot: I haz the power, 2014-04-24 09:20:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-24 09:20:54]

Try jobs failed on following builders:
  tryserver.chromium on chromium_presubmit

[jracle (use Gerrit), 2014-04-24 10:50:41]

The CQ bit was checked by jracle@logitech.com

[commit-bot: I haz the power, 2014-04-24 11:03:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jracle@logitech.com/240263003/80001

[commit-bot: I haz the power, 2014-04-24 16:27:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-04-24 16:27:30]

Try jobs failed on following builders:
  tryserver.chromium on linux_chromium_chromeos_rel

[Ken Rockot(use gerrit already), 2014-04-24 16:40:39]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-04-24 16:42:22]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/jracle@logitech.com/240263003/80001

[commit-bot: I haz the power, 2014-04-24 23:03:32]

Change committed as 266022