Migrate WebUSB LayoutTests into external/wpt

third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb-helpers.js

 'use strict';
 
+// To enable these tests Chrome must be run with these options:
+//
+//   --enable-blink-features=MojoJS,MojoJSTest
+let loadChromiumResources = Promise.resolve().then(() => {

[ortuno, 2017/06/21 00:43:50]

Sad that we have to include Chrome-specific code in cross-browser code.

[foolip, 2017/06/21 08:23:29]

Could this be made part of wpt_automation instead?

[Reilly Grant (use Gerrit), 2017/06/21 21:35:16]

One of the goals of this patch is to make these tests work in stable Chrome when
you navigate to web-platform-tests.org with the right flags set and that
requires including the Chromium-specific code in the WPT repository.

[foolip, 2017/06/22 22:30:38]

Hmm, I didn't notice that /resources/chromium/*.js were also added in wpt. I can
see how this way solves the problem, but I don't think there are other
vendor-specific bits in wpt right now, what other approaches are at all
possible?

Would it not be a problem that mojo mocking bits are versioned together with the
wpt and not Chromium? If I take the stable Chromium and /resources/chromium/*.js
have changed on master, I presume the tests may not work at all?

Does the --enable-blink-features=MojoJS,MojoJSTest bit already work with a plain
Chromium build? If it doesn't, and there's some other work to be done, would it
be OK to have this only work in content_shell until that work is done?

[foolip, 2017/06/22 22:52:18]

Discussed in https://codereview.chromium.org/2789723003/#msg59, if it's not
likely to be a practical problem then that's fine, if it is a problem then
revisit.
Apparently that already works.

If we're to run the tests with vanilla Chrome and vanilla WPT, then there are
only 3 options for where the mojo mocking scripts live:
1. In Chrome, but we'll not do that because of the binary size.
2. In WPT itself, as done here.
3. In a third thing which is versions together with Chrome and/or WPT, a bit
like ChromeDriver perhaps.

Option 3 would be more pure, but more work to get off the ground. And moving
from this to such a solution shouldn't be very disruptive.

Perhaps there will be some raised eyebrows in wpt, but I think I agree with the
choices now, and we can see how it goes. LGTM, thanks for your restless work on
this!

+  if (!MojoInterfaceInterceptor) {
+    // Do nothing on non-Chromium-based browsers or when the Mojo bindings are
+    // not present in the global namespace.
+    return;
+  }
+
+  let chain = Promise.resolve();
+  [
+    '/resources/chromium/mojo_bindings.js',
+    '/resources/chromium/device.mojom.js',
+    '/resources/chromium/device_manager.mojom.js',
+    '/resources/chromium/chooser_service.mojom.js',
+    '/resources/chromium/webusb-test.js',
+  ].forEach(path => {
+    let script = document.createElement('script');
+    script.src = path;
+    script.async = false;
+    chain = chain.then(() => new Promise(resolve => {
+      script.onload = () => resolve();
+    }));
+    document.head.appendChild(script);
+  });
+
+  return chain;
+});
+
 function usb_test(func, name, properties) {
   promise_test(async () => {
-    await navigator.usb.test.initialize()
+    await loadChromiumResources;
+    await navigator.usb.test.initialize();
     try {
       await func();
     } finally {
       await navigator.usb.test.reset();
     }
   }, name, properties);
 }
 
 // Returns a promise that is resolved when the next USBConnectionEvent of the
 // given type is received.
@@ skipping 49 matching lines @@
     } else if (Array.isArray(deviceInit[property])) {
       assert_equals(usbDevice[property].length, deviceInit[property].length);
       for (var i = 0; i < usbDevice[property].length; ++i)
         assertDeviceInfoEquals(usbDevice[property][i], deviceInit[property][i]);
     } else {
       assert_equals(usbDevice[property], deviceInit[property], property);
     }
   }
 }
 
-// TODO(reillyg): Remove when jyasskin upstreams this to testharness.js:
-// https://crbug.com/509058.
-function callWithKeyDown(functionCalledOnKeyPress) {
+function callWithTrustedClick(callback) {
   return new Promise(resolve => {
-    function onKeyPress() {
-      document.removeEventListener('keypress', onKeyPress, false);
-      resolve(functionCalledOnKeyPress());
-    }
-    document.addEventListener('keypress', onKeyPress, false);
-
-    eventSender.keyDown(' ', []);
+    let button = document.createElement('button');
+    button.textContent = 'click to continue test';
+    button.style.display = 'block';
+    button.style.fontSize = '20px';
+    button.style.padding = '10px';
+    button.onclick = () => {
+      resolve(callback());
+      document.body.removeChild(button);
+    };
+    document.body.appendChild(button);
   });
 }
-
-function runGarbageCollection() {
-  // Run gc() as a promise.
-  return new Promise((resolve, reject) => {
-    GCController.collect();
-    setTimeout(resolve, 0);
-  });
-}