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

Add missing worklet referrer tests #19165

Merged
merged 2 commits into from
Sep 25, 2019

Conversation

chromium-wpt-export-bot
Copy link
Collaborator

@chromium-wpt-export-bot chromium-wpt-export-bot commented Sep 19, 2019

This CL adds more web-platform-tests for the Referer on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrick@chromium.org, kouhei@chromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699351}

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.

Already reviewed downstream.

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrick@chromium.org, kouhei@chromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699351}
@foolip
Copy link
Member

foolip commented Sep 25, 2019

wpt-firefox-nightly-stability has failed on Taskcluster because of slow tests:

Test Result Longest duration (ms) Timeout (ms)
/worklets/animation-worklet-referrer.https.html OK 8358 10000
/worklets/audio-worklet-referrer.https.html OK 8613 10000
/worklets/layout-worklet-referrer.https.html OK 9774 10000

@zcorpan I think this is the first time I've seen this, but it's a good catch!

They take close to 10 seconds on Firefox:
#19165 (comment)
@chromium-wpt-export-bot chromium-wpt-export-bot merged commit eaa5519 into master Sep 25, 2019
@chromium-wpt-export-bot chromium-wpt-export-bot deleted the chromium-export-cl-1813122 branch September 25, 2019 12:27
@zcorpan
Copy link
Member

zcorpan commented Sep 26, 2019

Yay!

moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request Sep 26, 2019
…stonly

Automatic update from web-platform-tests
Add missing worklet referrer tests

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrick@chromium.org, kouhei@chromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699351}

--
Add long timeout to worklet referrer tests

They take close to 10 seconds on Firefox:
web-platform-tests/wpt#19165 (comment)

--

wpt-commits: 98c89472a9e6359bfac5b40b05188c9c5b39be79, eaa551911f12c5c526f56cafd0c7ab24667ead3c
wpt-pr: 19165
xeonchen pushed a commit to xeonchen/gecko that referenced this pull request Sep 27, 2019
…stonly

Automatic update from web-platform-tests
Add missing worklet referrer tests

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrick@chromium.org, kouhei@chromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrick@chromium.org>
Commit-Queue: Dominic Farolino <dom@chromium.org>
Cr-Commit-Position: refs/heads/master@{#699351}

--
Add long timeout to worklet referrer tests

They take close to 10 seconds on Firefox:
web-platform-tests/wpt#19165 (comment)

--

wpt-commits: 98c89472a9e6359bfac5b40b05188c9c5b39be79, eaa551911f12c5c526f56cafd0c7ab24667ead3c
wpt-pr: 19165
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request Oct 4, 2019
…stonly

Automatic update from web-platform-tests
Add missing worklet referrer tests

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrickchromium.org, kouheichromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrickchromium.org>
Commit-Queue: Dominic Farolino <domchromium.org>
Cr-Commit-Position: refs/heads/master{#699351}

--
Add long timeout to worklet referrer tests

They take close to 10 seconds on Firefox:
web-platform-tests/wpt#19165 (comment)

--

wpt-commits: 98c89472a9e6359bfac5b40b05188c9c5b39be79, eaa551911f12c5c526f56cafd0c7ab24667ead3c
wpt-pr: 19165

UltraBlame original commit: 4b1a16410004641274faf37bbb27a3fca5013a56
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request Oct 4, 2019
…stonly

Automatic update from web-platform-tests
Add missing worklet referrer tests

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrickchromium.org, kouheichromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrickchromium.org>
Commit-Queue: Dominic Farolino <domchromium.org>
Cr-Commit-Position: refs/heads/master{#699351}

--
Add long timeout to worklet referrer tests

They take close to 10 seconds on Firefox:
web-platform-tests/wpt#19165 (comment)

--

wpt-commits: 98c89472a9e6359bfac5b40b05188c9c5b39be79, eaa551911f12c5c526f56cafd0c7ab24667ead3c
wpt-pr: 19165

UltraBlame original commit: 4b1a16410004641274faf37bbb27a3fca5013a56
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request Oct 5, 2019
…stonly

Automatic update from web-platform-tests
Add missing worklet referrer tests

This CL adds more web-platform-tests for the `Referer` on worklet
descendant requests. This CL introduces tests where we fetch a
same-origin descendant script from a cross-origin worklet script served
with varying referrer policies.

The tests are similar to those introduced in crrev.com/c/1809205 for
module scripts, and will pass once crrev.com/c/1808663 is merged.

R=ikilpatrickchromium.org, kouheichromium.org

Bug: 786862,1004083
Change-Id: I4425be4264424b52e4ec307513a404541869f2a1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1813122
Reviewed-by: Ian Kilpatrick <ikilpatrickchromium.org>
Commit-Queue: Dominic Farolino <domchromium.org>
Cr-Commit-Position: refs/heads/master{#699351}

--
Add long timeout to worklet referrer tests

They take close to 10 seconds on Firefox:
web-platform-tests/wpt#19165 (comment)

--

wpt-commits: 98c89472a9e6359bfac5b40b05188c9c5b39be79, eaa551911f12c5c526f56cafd0c7ab24667ead3c
wpt-pr: 19165

UltraBlame original commit: 4b1a16410004641274faf37bbb27a3fca5013a56
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.

6 participants