Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PlzDedicatedWorker: WPT for clients.matchAll() with a blob URL worker #27197

Merged
merged 1 commit into from Jan 15, 2021

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Jan 15, 2021

The new test checks if clients.matchAll() can detect a blob URL worker
client. The test passed when the PlzDedicatedWorker feature is enabled
and fails when it is disabled.

Bug: 1017034
Change-Id: Ib5b0916c2b9a4068b379769236bd0da2c6285a5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629330
Commit-Queue: Asami Doi <asamidoi@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844025}

Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The review process for this patch is being conducted in the Chromium project.

@chromium-wpt-export-bot chromium-wpt-export-bot force-pushed the chromium-export-cl-2629330 branch 2 times, most recently from e93e4c7 to cca4c51 Compare January 15, 2021 10:27
The new test checks if `clients.matchAll()` can detect a blob URL worker
client. The test passed when the PlzDedicatedWorker feature is enabled
and fails when it is disabled.

Bug: 1017034
Change-Id: Ib5b0916c2b9a4068b379769236bd0da2c6285a5a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2629330
Commit-Queue: Asami Doi <asamidoi@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#844025}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants