Closed Bug 1463039 Opened 2 years ago Closed 2 years ago

[wpt-sync] Sync PR 11085 - Add referrer policy support to script fetching


(Core :: DOM: Security, enhancement, P4)




Tracking Status
firefox63 --- fixed


(Reporter:, Unassigned)




(Whiteboard: [wptsync downstream][domsecurity-active])

Sync web-platform-tests PR 11085 into mozilla-central (this bug is closed when the sync is complete).

Details from upstream follow.

Dominic Farolino <> wrote:
>  Add referrer policy support to script fetching
>  This adds referrer policy attribute to top-level script
>  elements and ensures that the referrer policy for a script
>  specified by its attribute gets inherited by imported script
>  subresources.
>  Intent to Ship:!topic/blink-dev/jfd4knvCiio
>  Bug: 841673
>  Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
>  Change-Id: I49650d593ba433a53793c846afc18be83dc9fc87
>  Reviewed-on:
>  WPT-Export-Revision: 37e5d06f88bc1ad1cbe52aadec34ac2417c6a850
Component: web-platform-tests → DOM: Security
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]
Try push failed: decision task returned error
I was idle for several months and I am lost what wpt-sync is doing.
Hi James, I would like to sync referrer policy-script tests (tests for bug 1460920), so I have to manually update wpt tests and meta in this bug right?
Flags: needinfo?(james)
No, you do nothing, it should work automatically (although we hit a few bumps last month so we are a little behind). The problem here, however is that the upstream PR is failing travis and hasn't merged. Looks like it's just timeouts so I can override that and get the things landed.
Flags: needinfo?(james)
Failed to get results from try push
Thanks James,
Hmm, I see the error message is "standard_init_linux.go:195: exec user process caused "argument list too long".
Is that the try command is too long?

try: -b do -p win32,win64,linux64,linux -u web-platform-tests-e10s-1[linux64-stylo,Ubuntu,10.10,Windows 10],web-platform-tests-1[linux64-stylo,Ubuntu,10.10,Windows 10] -t none --artifact --try-test-paths web-platform-tests:testing/web-platform/tests/referrer-policy/css-integration/external-import-stylesheet.html web-platform-tests:testing/web-platform/tests/referrer-policy/css-integration/external-stylesheet.html web-platform-tests:testing/web-platform/tests/referrer-policy/css-integration/inline-style.html web-platform-tests:testing/web-platform/tests/referrer-policy/css-integration/internal-import-stylesheet.html web-platform-tests:testing/web-platform/tests/referrer-policy/css-integration/internal-stylesheet.html web-platform-.....................
(over 65550 characters?)
Do you have any idea about that?
Flags: needinfo?(james)
Uh, yeah, that's a known problem :/ Eventually we will probably move to using try_tasks.json for the try push, which might solve the problem. For now maybe I should add a hack to reduce the lenght of the paths.
Flags: needinfo?(james)
Blocks: 1460920
Thanks James
Whiteboard: [wptsync downstream][domsecurity-backlog] → [wptsync downstream][domsecurity-active]
Do we have any update for this? Did we file a bug to do a hack?
Thanks James for looking at this bug.
Flags: needinfo?(james)
Pushed by
[wpt PR 11085] - Add referrer policy support to script fetching, a=testonly
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
This landed now, so the tests should be available.
Flags: needinfo?(james)
Thanks James :)
You need to log in before you can comment on or make changes to this bug.