Support permission_broker for chrome.hid devices.

Support permission_broker for chrome.hid devices.

This enables the Chrome to query permission_broker for access
to hidraw device paths when running on Chrome OS.

HidService has been refactored so that it is owned by the
extensions API embedder, rather than being a singleton.

Because hid has just moved to //extensions as well, this also establishes a new ShellExtensionsAPIClient to create a HidService for app_shell.

BUG=367145
TEST=manual test on ChromeOS dev build
TBR=derat@chromium.org for mechanical app_shell changes

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

[Ken Rockot(use gerrit already), 2014-07-25 18:45:36]

Renaud, could you please take a look?

[rpaquay, 2014-07-25 19:51:41]

lgtm

[Ken Rockot(use gerrit already), 2014-07-25 19:52:49]

+reillyg@ FYI

[Ken Rockot(use gerrit already), 2014-07-25 19:52:53]

The CQ bit was checked by rockot@chromium.org

[Reilly Grant (use Gerrit), 2014-07-25 19:57:09]

lgtm

https://codereview.chromium.org/419713002/diff/40001/chrome/browser/extension...
File chrome/browser/extensions/api/hid/hid_device_manager.cc (left):

https://codereview.chromium.org/419713002/diff/40001/chrome/browser/extension...
chrome/browser/extensions/api/hid/hid_device_manager.cc:6: 
Leave this whitespace.

[commit-bot: I haz the power, 2014-07-25 19:57:45]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rockot@chromium.org/419713002/40001

[Ken Rockot(use gerrit already), 2014-07-25 20:02:06]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-07-25 20:04:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rockot@chromium.org/419713002/60001

[Ken Rockot(use gerrit already), 2014-07-26 00:17:07]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2014-07-26 05:00:15]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/rockot@chromium.org/419713002/150001

[commit-bot: I haz the power, 2014-07-26 18:25:43]

Change committed as 285774

[tapted, 2014-07-28 00:04:50]

On 2014/07/26 18:25:43, I haz the power (commit-bot) wrote:
> Change committed as 285774

Lnux ASan LSan Tests (1) is detecting a memory leak since
http://build.chromium.org/p/chromium.memory/builders/Linux%20ASan%20LSan%20Te...

errors like

Direct leak of 72 byte(s) in 1 object(s) allocated from:
    #0 0x44145b in operator new(unsigned long)
/usr/local/google/work/chromium/src/third_party/llvm/projects/compiler-rt/lib/asan/asan_new_delete.cc:55
    #1 0x5e8afd in
device::HidService::Create(scoped_refptr\u003Cbase::MessageLoopProxy>)
device/hid/hid_service.cc:26
    #2 0x509324 in device::HidServiceTest_Create_Test::TestBody()
device/hid/hid_service_unittest.cc:15

Looks like a simple fix - https://codereview.chromium.org/424623002/ is in the
CQ.