Closed Bug 1656759 Opened 5 years ago Closed 5 years ago

[wpt-sync] Sync PR 24845 - [WPT] Add tests for sandbox directive for shared/service workers

Categories

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

task

Tracking

()

RESOLVED FIXED
82 Branch
Tracking Status
firefox82 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

()

Details

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

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

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

Hiroshige Hayashizaki <hiroshige@chromium.org> wrote:

[WPT] Add tests for sandbox directive for shared/service workers

These tests confirm sandbox directives are no-op for workers,
because sandbox directive is not yet specified for workers.

(This CL doesn't add tests for dedicated workers, because
dedicated workers inherit parent Document's CSP,
the parent Documents should be sandboxed and thus it's hard
to test whether dedicated workers from sandboxed Documents
are sandboxed)

Bug: 1112143, https://github.com/w3c/webappsec-csp/issues/279
Change-Id: Iff34da98de61214584681ae6f12ffcbf1dee1880

Reviewed-on: https://chromium-review.googlesource.com/2333908
WPT-Export-Revision: e95a50bb1266fdc377a5a8c1368cead18b47d593

Component: web-platform-tests → DOM: Security
Product: Testing → Core
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]

CI Results

Ran 12 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 1 subtests

Status Summary

Firefox

OK : 2
PASS: 9

Chrome

OK : 2
PASS: 3
FAIL: 6

Safari

OK : 2
PASS: 8
FAIL: 1

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Pushed by wptsync@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/03089e59305c [wpt PR 24845] - [WPT] Add tests for sandbox directive for shared/service workers, a=testonly
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → 82 Branch
You need to log in before you can comment on or make changes to this bug.