Add onconnect and ondisconnect events to navigator.usb.

Add onconnect and ondisconnect events to navigator.usb.

This patch updates USB.idl to add these two events, making it an
EventTarget. WebUSBClient still needs to be updated to support
delivering these events to Blink.

BUG=492204
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=200912

[Reilly Grant (use Gerrit), 2015-08-19 20:26:32]

reillyg@chromium.org changed reviewers:
+ rockot@chromium.org

[Reilly Grant (use Gerrit), 2015-08-19 20:26:33]

Please take a look.

[Ken Rockot(use gerrit already), 2015-08-19 21:46:26]

lgtm

[Reilly Grant (use Gerrit), 2015-08-19 21:48:02]

reillyg@chromium.org changed reviewers:
+ haraken@chromium.org

[Reilly Grant (use Gerrit), 2015-08-19 21:48:17]

Adding haraken@ for Source/modules/EventTargetModulesFactory.in.

[haraken, 2015-08-20 00:31:31]

haraken@chromium.org changed reviewers:
+ tkent@chromium.org

[haraken, 2015-08-20 00:31:31]

LGTM

+tkent-san for API owner review on EventTargetModules.in.

https://codereview.chromium.org/1291753004/diff/1/Source/modules/webusb/USB.cpp
File Source/modules/webusb/USB.cpp (right):

https://codereview.chromium.org/1291753004/diff/1/Source/modules/webusb/USB.c...
Source/modules/webusb/USB.cpp:114: return m_controller->frame()->document();

m_controller->frame() can be null if the frame was detached.

  return m_controller->frame() ? m_controller->frame()->document() : nullptr;

[tkent, 2015-08-20 00:44:40]

API owner review is unnecessary because USB API is not shipped yet.
anyway lgtm if layout tests are fixed.

[Reilly Grant (use Gerrit), 2015-08-20 01:24:32]

The CQ bit was checked by reillyg@chromium.org

[Reilly Grant (use Gerrit), 2015-08-20 01:24:32]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, rockot@chromium.org, tkent@chromium.org
Link to the patchset: https://codereview.chromium.org/1291753004/#ps40001
(title: "Update LayoutTests.")

[commit-bot: I haz the power, 2015-08-20 01:24:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291753004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291753004/40001

[commit-bot: I haz the power, 2015-08-20 03:04:58]

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

[commit-bot: I haz the power, 2015-08-20 03:04:59]

Try jobs failed on following builders:
  linux_chromium_gn_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Reilly Grant (use Gerrit), 2015-08-20 16:49:57]

The CQ bit was checked by reillyg@chromium.org

[commit-bot: I haz the power, 2015-08-20 16:50:09]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1291753004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1291753004/40001

[commit-bot: I haz the power, 2015-08-20 17:32:59]

Committed patchset #3 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=200912