New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 841510 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 12
Cc:
Components:
EstimatedDays: 21
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Task

Blocked on:
issue 852878

Blocking:
issue 837406



Sign in to add a comment

Create Web Platform Tests for WebUSB on Web Workers

Project Member Reported by odejesush@chromium.org, May 9 2018

Issue description

Web Platform Tests should be created to ensure that WebUSB works properly inside of the supported workers.
 
Project Member

Comment 1 by bugdroid1@chromium.org, May 21 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/3267532a0e80b0aeb73ec4e72434df630bc9ce63

commit 3267532a0e80b0aeb73ec4e72434df630bc9ce63
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Mon May 21 23:10:58 2018

Create WebUSB idlharness test for workers

This change creates an idlharness test for the workers to ensure that
all the interfaces are exposed in the worker. This change also updates
the webusb.idl file to match the exposure of the interfaces.

Bug:  841510 
Change-Id: Iad72f242ee0cdbf9a3057828bd4bd5542f6fe063
Reviewed-on: https://chromium-review.googlesource.com/1053063
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Chong Zhang <chongz@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560399}
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/external/wpt/interfaces/webusb.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.html
[add] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/idlharness.dedicatedworker.sharedworker.js
[add] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.dedicatedworker.https.html
[add] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.sharedworker.https.html
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-dedicated-worker-expected.txt
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-shared-worker-expected.txt
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_alternate_interface.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_configuration.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_connection_event.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_device.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_endpoint.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_in_transfer_result.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_interface.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_isochronous_in_transfer_packet.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_isochronous_in_transfer_result.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_isochronous_out_transfer_packet.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_isochronous_out_transfer_result.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/usb_out_transfer_result.idl
[modify] https://crrev.com/3267532a0e80b0aeb73ec4e72434df630bc9ce63/third_party/blink/renderer/modules/webusb/worker_navigator_usb.idl

Project Member

Comment 2 by bugdroid1@chromium.org, Jun 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1ff37758f07612906228581dbc808df4b282e250

commit 1ff37758f07612906228581dbc808df4b282e250
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Wed Jun 06 18:44:44 2018

Multi-global WebUSB WPT for window and worker

This change updates the existing web platform tests for WebUSB to use
multi-global tests to test the API in the supported contexts. The
change updates the following tests:
* idlharness
* usb
* usbConnectionEvent

Bug:  841510 
Change-Id: I6b118aa4121109cd6a48054c2ebc69b831660afc
Reviewed-on: https://chromium-review.googlesource.com/1087209
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#564981}
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.js
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.html
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.any.js
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.window.js
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.https-expected.txt
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.https.html
[rename] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.js
[add] https://crrev.com/1ff37758f07612906228581dbc808df4b282e250/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.any.js
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.html
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.dedicatedworker.https.html
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.sharedworker.https.html
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.dedicatedworker.https.html
[delete] https://crrev.com/dd6a1c05d08ccb6e44ef144007d4a4167a3b8464/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.sharedworker.https.html

Project Member

Comment 3 by bugdroid1@chromium.org, Jun 6 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/43e6203f6d76c91636cfc097a27fc30fa7b6a26c

commit 43e6203f6d76c91636cfc097a27fc30fa7b6a26c
Author: Kevin McNee <mcnee@chromium.org>
Date: Wed Jun 06 20:26:22 2018

Revert "Multi-global WebUSB WPT for window and worker"

This reverts commit 1ff37758f07612906228581dbc808df4b282e250.

Reason for revert: external/wpt/webusb/idlharness.https.any.* tests are failing

Original change's description:
> Multi-global WebUSB WPT for window and worker
> 
> This change updates the existing web platform tests for WebUSB to use
> multi-global tests to test the API in the supported contexts. The
> change updates the following tests:
> * idlharness
> * usb
> * usbConnectionEvent
> 
> Bug:  841510 
> Change-Id: I6b118aa4121109cd6a48054c2ebc69b831660afc
> Reviewed-on: https://chromium-review.googlesource.com/1087209
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564981}

TBR=reillyg@chromium.org,chongz@chromium.org,odejesush@chromium.org

Change-Id: Ifde97a1736134477422b6fd81392b11bbd775a63
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug:  841510 
Reviewed-on: https://chromium-review.googlesource.com/1089690
Reviewed-by: Kevin McNee <mcnee@chromium.org>
Commit-Queue: Kevin McNee <mcnee@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565022}
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.html
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/idlharness.dedicatedworker.sharedworker.js
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb.dedicatedworker.sharedworker.js
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb.serviceworker.js
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.html
[delete] https://crrev.com/d2cb3bc963a7b2d472f99297425b28be1175a77c/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.any.js
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.html
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.dedicatedworker.https.html
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.sharedworker.https.html
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.dedicatedworker.https.html
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.serviceworker.https-expected.txt
[rename] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.serviceworker.https.html
[add] https://crrev.com/43e6203f6d76c91636cfc097a27fc30fa7b6a26c/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.sharedworker.https.html

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 8 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/03ef65db57d1f9932cbfca7cc1910efcdf07acdb

commit 03ef65db57d1f9932cbfca7cc1910efcdf07acdb
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Fri Jun 08 23:44:18 2018

Update existing WebUSB WPT to be multi-global

This change updates the remaining WebUSB web platform tests to be
multi-global so that the API is also tested in the worker contexts.

Bug:  841510 
Change-Id: I9df24f1ae4289ab742db275c34552200bb477b22
Reviewed-on: https://chromium-review.googlesource.com/1089033
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565778}
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbDevice.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbInTransferResult.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousInTransferPacket.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousInTransferResult.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousOutTransferPacket.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousOutTransferResult.https.any.js
[rename] https://crrev.com/03ef65db57d1f9932cbfca7cc1910efcdf07acdb/third_party/WebKit/LayoutTests/external/wpt/webusb/usbOutTransferResult.https.any.js

Project Member

Comment 5 by bugdroid1@chromium.org, Jun 14 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/89102421458afea3981c12738ccf6f1231a95b79

commit 89102421458afea3981c12738ccf6f1231a95b79
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Thu Jun 14 16:15:11 2018

Reland "Multi-global WebUSB WPT for window and worker"

This is a reland of 1ff37758f07612906228581dbc808df4b282e250

Original change's description:
> Multi-global WebUSB WPT for window and worker
>
> This change updates the existing web platform tests for WebUSB to use
> multi-global tests to test the API in the supported contexts. The
> change updates the following tests:
> * idlharness
> * usb
> * usbConnectionEvent
>
> Bug:  841510 
> Change-Id: I6b118aa4121109cd6a48054c2ebc69b831660afc
> Reviewed-on: https://chromium-review.googlesource.com/1087209
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#564981}

Bug:  841510 
Change-Id: Ie354fdd54a4cd94a4801ee634c8d3263e0aa6e93
Reviewed-on: https://chromium-review.googlesource.com/1089647
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Chong Zhang <chongz@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#567290}
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any-expected.txt
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.js
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.sharedworker-expected.txt
[copy] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.worker-expected.txt
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.html
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.any.js
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.window.js
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.https-expected.txt
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.https.html
[rename] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.serviceworker.js
[add] https://crrev.com/89102421458afea3981c12738ccf6f1231a95b79/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.any.js
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.html
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.dedicatedworker.https.html
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.sharedworker.https-expected.txt
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/idlharness.sharedworker.https.html
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.dedicatedworker.https.html
[delete] https://crrev.com/1ae77fbaea90159a2545ee682d3a57010e13eb70/third_party/WebKit/LayoutTests/external/wpt/webusb/worker/usb.sharedworker.https.html

Blockedon: 852878
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 18 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2

commit 44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Mon Jun 18 21:33:45 2018

Add WebUSB FeaturePolicy WPT for Workers

This change adds Feature Policy tests for Dedicated Workers.

Bug:  841510 
Change-Id: I117f8a5bb61d5783b6867e7edcc9d99aff41a497
Reviewed-on: https://chromium-review.googlesource.com/1093379
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#568170}
[add] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/feature-policy/resources/feature-policy-usb-worker.html
[add] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/feature-policy/resources/feature-policy-usb-worker.js
[add] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb-allowed-by-feature-policy-worker.js
[add] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb-disabled-by-feature-policy-worker.js
[modify] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/usb-allowed-by-feature-policy-attribute-redirect-on-load.https.sub.html
[modify] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/usb-allowed-by-feature-policy-attribute.https.sub.html
[modify] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/usb-allowed-by-feature-policy.https.sub.html
[modify] https://crrev.com/44d285343d2af0fbea64f6d9f5d44b0e5f4c18f2/third_party/WebKit/LayoutTests/external/wpt/webusb/usb-disabled-by-feature-policy.https.sub.html

Project Member

Comment 8 by bugdroid1@chromium.org, Jun 25 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1360c1e0192c81e22a2fa629d2636c3699fbdf9a

commit 1360c1e0192c81e22a2fa629d2636c3699fbdf9a
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Mon Jun 25 22:25:24 2018

Add WebUSB iframe reference disconnect test

This change adds a test for the case when a reference to a WebUSB object
from an iframe is attempted to be used after the iframe has
disconnected.

Bug:  841510 
Change-Id: I8b8d5f5d50af6d062bdc3cab9b516f598bcf8fae
Reviewed-on: https://chromium-review.googlesource.com/1112346
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Chong Zhang <chongz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#570203}
[add] https://crrev.com/1360c1e0192c81e22a2fa629d2636c3699fbdf9a/third_party/WebKit/LayoutTests/usb/usb-iframe.html

Status: Fixed (was: Assigned)
I have converted all save for a few tests into multiglobal tests. The remaining tests that were not converted only test cases specific to the window context.
Project Member

Comment 10 by bugdroid1@chromium.org, Jul 26

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/408f60caa64af7ae037c12dcb9ca592768676c39

commit 408f60caa64af7ae037c12dcb9ca592768676c39
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Thu Jul 26 23:17:51 2018

Remove Shared Worker WPT for WebUSB

This change removes the WebUSB on Shared Worker web platform tests,
since the feature is not yet implemented in any browser.

Bug:  841510 
Change-Id: I6cd09c245edcd69799b39701b82ba1510caf845e
Reviewed-on: https://chromium-review.googlesource.com/1151752
Reviewed-by: Chong Zhang <chongz@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Cr-Commit-Position: refs/heads/master@{#578479}
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.js
[delete] https://crrev.com/f59cc3646d251a75fa344dcb079b56c185482b13/third_party/WebKit/LayoutTests/external/wpt/webusb/idlharness.https.any.sharedworker-expected.txt
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/insecure-context.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usb.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbConnectionEvent.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbDevice.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbInTransferResult.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousInTransferPacket.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousInTransferResult.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousOutTransferPacket.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbIsochronousOutTransferResult.https.any.js
[modify] https://crrev.com/408f60caa64af7ae037c12dcb9ca592768676c39/third_party/WebKit/LayoutTests/external/wpt/webusb/usbOutTransferResult.https.any.js

Project Member

Comment 11 by bugdroid1@chromium.org, Aug 28

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/edfc097afbe8d1f3917fa2d42738bdac34bfe00c

commit edfc097afbe8d1f3917fa2d42738bdac34bfe00c
Author: Ovidio Henriquez <odejesush@chromium.org>
Date: Tue Aug 28 23:58:27 2018

Update WebUSB terminate child context tests

This change updates tests for WebUSB where a child context is terminated
while having a USB device open. The existing tests are modified to send
the MojoHandle intercepted by MojoInterfaceInterceptor through a
MessageChannel to allow a new test for a worker context to be created.
As a result, the MojoHandle class is made transferable over a
MessageChannel.

Bug:  841510 ,  852878 
Change-Id: Ib7f74d172b9dcc16a4821a4063cfffc196466a88
Reviewed-on: https://chromium-review.googlesource.com/1162351
Commit-Queue: Ovidio Henriquez <odejesush@chromium.org>
Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#586942}
[add] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/resources/chromium/webusb-child-test.js
[add] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/resources/chromium/webusb-child-test.js.headers
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/resources/chromium/webusb-test.js
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/open-in-iframe.html
[add] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/open-in-worker.js
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/webusb/resources/usb-helpers.js
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/webusb/usbDevice-iframe.https.html
[add] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/external/wpt/webusb/usbDevice-worker.https.html
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/WebKit/LayoutTests/usb/usbDevice-iframe.html
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/serialization_tag.h
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/serialized_script_value.cc
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/serialized_script_value.h
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/transferables.h
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/v8_script_value_deserializer.cc
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/v8_script_value_serializer.cc
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/bindings/core/v8/serialization/v8_script_value_serializer_test.cc
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/core/messaging/message_port.cc
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/core/messaging/message_port.h
[modify] https://crrev.com/edfc097afbe8d1f3917fa2d42738bdac34bfe00c/third_party/blink/renderer/core/mojo/mojo_handle.h

Sign in to add a comment