diff options
| author | reillyg <reillyg@chromium.org> | 2016-03-03 17:13:36 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-03-04 01:15:44 +0000 |
| commit | ee8bc708fdfe52c34d39f5cc9a8407fe9cfb459c (patch) | |
| tree | 6a43df82d61f65288bb1ae9632d73a08bf72fd0b /sql | |
| parent | d285b019189141a8c235f9eed8764d6129043380 (diff) | |
| download | chromium_src-ee8bc708fdfe52c34d39f5cc9a8407fe9cfb459c.zip chromium_src-ee8bc708fdfe52c34d39f5cc9a8407fe9cfb459c.tar.gz chromium_src-ee8bc708fdfe52c34d39f5cc9a8407fe9cfb459c.tar.bz2 | |
Check USB device permissions with a full device::usb::DeviceInfo.
UsbChooserContext::HasDevicePermission was originally written to take
the device GUID. The problem with this is that it means that the
device::UsbDevice object has to be looked up based on that GUID and if
the device has been disconnected (as happens when trying to fire the
navigator.usb.ondisconnect event) then that lookup fails and so the
permission check always fails. Thus, no events are delivered. Instead,
since WebUsbPermissionProvider has it anyways, HasDevicePermission can
take the DeviceInfo object representing the device which has enough
information to do a permission check even if the device has been
disconnected.
BUG=None
Review URL: https://codereview.chromium.org/1747543002
Cr-Commit-Position: refs/heads/master@{#379161}
Diffstat (limited to 'sql')
0 files changed, 0 insertions, 0 deletions