1578527 - [wpt-sync] Sync PR 18812 - [DO NOT REVIEW] Move websocket-request,picture-tag,img-tag,classic-data-worker-fetch,fetch-request,video-tag,object-tag,worker-request,link-prefetch-tag,module-worker-top-level,audio-tag,beacon-request,module-data-worker-import…

Status: RESOLVED FIXED

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

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

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

PR: https://github.com/web-platform-tests/wpt/pull/18812
Details from upstream follow.

Hiroshige Hayashizaki <[e-mail]> wrote:

[DO NOT REVIEW] Move websocket-request,picture-tag,img-tag,classic-data-worker-fetch,fetch-request,video-tag,object-tag,worker-request,link-prefetch-tag,module-worker-top-level,audio-tag,beacon-request,module-data-worker-import,script-tag,link-css-tag (1)

fix

revert

fix lint

format

[WPT/common/security-features] Support nested source contexts

To enable spec.src.json to specify (possibly nested) source contexts,
this CL adds source_context_list_schema that represents
source contexts from where subresource requests are sent,
and how policies are delivered (by source contexts or subresource requests).
This CL adds source_context_list field to each test_expansion entry,
of which value is the key of source_context_list_schema to be used.

This CL adds source_context_schema and subresource_schema
that represent supported delivery types and subresources
for each source context or subresource type. These are used:

• To filter out test files for unsupported combinations of delivery types,
  source contexts and subresources.
  A part of the filtering was removed in
  https://chromium-review.googlesource.com/c/chromium/src/+/1717535
  and this CL re-adds it (in a different form of implementation).
• To determine what delivery types should be used for anotherPolicy
  placeholder.

After this CL, the generate.py generates source_context_list and
subresource_policy_deliveries fields that are passed directly
without coversion.
Specifically,

• The conversion from classic-data-worker-fetch to
  classic-data-worker + fetch-request in mixed-content-test-case.js
  is removed.
• \<iframe srcdoc>-related tests for img-tag referrer-policy tests
  are split into separate tests, and related code in
  referrer-policy-test-case.sub.js is removed.
  Instead, to support tests inside \<iframe srcdoc>, this CL adds
  referrerSource calculation that can support \<iframe srcdoc>s.

This CL also changes the structure of generated file names.
Now all files are generated under a dedicated directory gen/.

This CL creates test-case.sub.js instead of modifying
referrer-policy-test-case.sub.js etc., because:
After this CL, I'll commit new generated test HTML files,
incrementally in order to monitor potential regressions,
Because thousands of files are modified, split, and renamed.
During the incremental commits, test-case.sub.js is used
by new generated tests while
referrer-policy-test-case.sub.js etc. are still used by
old tests.
After all commits are done, referrer-policy-test-case.sub.js
etc. will be removed.

test-case.sub.js reflects renaming of JSON keys and values done
in previous commits.

• opt_in_method -> delivery_type/value
  https://chromium-review.googlesource.com/c/chromium/src/+/1717107
• delivery_method -> delivery_type
  https://chromium-review.googlesource.com/c/chromium/src/+/1716109
• referrer_policy -> delivery_value
  https://chromium-review.googlesource.com/c/chromium/src/+/1717592
• source_protocol -> source_scheme
  https://chromium-review.googlesource.com/c/chromium/src/+/1717115
  https://chromium-review.googlesource.com/c/chromium/src/+/1718991
• target_protocol + origin -> origin
  https://chromium-review.googlesource.com/c/chromium/src/+/1717470
• Use shorter values for origin and redirection
  https://chromium-review.googlesource.com/c/chromium/src/+/1717475
• referrer_url -> expectation
  https://chromium-review.googlesource.com/c/chromium/src/+/1717319

Bug: 906850
Change-Id: Ia6695081e5a93355c29fb1bdb7e89ba6b673261f

Remove fetch-request guard and fix typos

revert sub

Add more validation + rename

more

more

Reorder path

Change-Id: Ice864d30c7b173a1c863b2e0dd40b351e34173da
Reviewed-on: https://chromium-review.googlesource.com/1782135
WPT-Export-Revision: 99f4b71079dbded5e982f59dcc5962cf5c5e9fe1

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=297dd5158c95d2cdc38b2ce8d5db6b32f7364a5c

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=bd9b34272bb5577c42177bf2874cb442d8133e0e

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=5860d37a8b8e8954e1076e945efa75b0a3b77ab4

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=67df8a2a300000f34d826462343b09cd90f92ee2

Comment 5

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=8f7e05bc19bbd2fe3d24a897868da56b91db9474

Comment 6

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Ran 7 tests and 8 subtests
OK     : 7
PASS   : 7
FAIL   : 1

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/2aaca64b263f
[wpt PR 18812] - [WPT/common/security-features] Support nested source contexts, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/db58c82ab353
[wpt PR 18812] - Update wpt metadata, a=testonly

Comment 8

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/2aaca64b263f
https://hg.mozilla.org/mozilla-central/rev/db58c82ab353