Closed Bug 1639505 Opened 5 years ago Closed 5 years ago

2 /content-security-policy/inheritance/ tests are expected TIMEOUT

Categories

(Core :: DOM: Security, defect, P2)

defect

Tracking

()

RESOLVED FIXED
mozilla78
Tracking Status
firefox78 --- fixed

People

(Reporter: jmaher, Assigned: ckerschb)

References

Details

(Whiteboard: [domsecurity-active], [wptsync upstream])

Attachments

(1 file)

these 2 tests are marked expected timeout:
/content-security-policy/inheritance/document-write-iframe.html
/content-security-policy/inheritance/window.html

running locally, I see:
/content-security-policy/inheritance/document-write-iframe.html has a harness timeout.

/content-security-policy/inheritance/window.html has in the console:
Timeout window.open() inherits policy. Test timed out

:ckerschb, here is another one for you to triage

Flags: needinfo?(ckerschb)

(In reply to Joel Maher ( :jmaher ) (UTC-4) from comment #0)

these 2 tests are marked expected timeout:
/content-security-policy/inheritance/document-write-iframe.html

We are blocking correctly, but again something is off with document.write() and I can't figure what it is.

/content-security-policy/inheritance/window.html

window.open() inheritance should work. The test passes if I assign img.src before appendChild, basiccaly swapping those two lines:
https://searchfox.org/mozilla-central/source/testing/web-platform/tests/content-security-policy/inheritance/window.html#26-27

Will upload patch in a minute.

Assignee: nobody → ckerschb
Severity: -- → S3
Status: NEW → ASSIGNED
Flags: needinfo?(ckerschb)
Priority: -- → P2
Whiteboard: [domsecurity-active]
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/23797 for changes under testing/web-platform/tests
Whiteboard: [domsecurity-active] → [domsecurity-active], [wptsync upstream]
Upstream web-platform-tests status checks passed, PR will merge once commit reaches central.
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla78
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: