Closed
Bug 1490706
Opened 8 years ago
Closed 8 years ago
[wpt-sync] Sync PR 12969 - Element::setAttributeNS() modified to accept TrustedTypes
Categories
(Core :: DOM: Security, enhancement, P4)
Core
DOM: Security
Tracking
()
RESOLVED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream][domsecurity-backlog])
Sync web-platform-tests PR 12969 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/12969
Details from upstream follow.
Maja Kabus <kabusm@google.com> wrote:
> Element::setAttributeNS() modified to accept TrustedTypes
>
> Added new implementation of Element::setAttributeNS() to accept
> generic TrustedType as an argument.
> Existing implementation renamed to Element::setAttributeNSInternal
> and made private.
>
> Bug: 739170
> Change-Id: Ib83250f579ba63fc519d1870671b2801c2ce2c5c
> Reviewed-on: https://chromium-review.googlesource.com/1221321
> WPT-Export-Revision: f9ea08b2e6cf158acb4901e4988df833c2e0fbb5
| Assignee | ||
Updated•8 years ago
|
Component: web-platform-tests → DOM: Security
Product: Testing → Core
| Assignee | ||
Comment 1•8 years ago
|
||
| Assignee | ||
Comment 2•8 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f458e99ec42f5f151d87435d75dd37318e54d48f
Updated•8 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]
| Assignee | ||
Updated•8 years ago
|
Whiteboard: [wptsync downstream][domsecurity-backlog] → [wptsync downstream]
| Assignee | ||
Comment 3•8 years ago
|
||
| Assignee | ||
Comment 4•8 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=d7a03e75aa03f399515692c7666b3f63a2774f4f
Updated•8 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]
| Assignee | ||
Updated•8 years ago
|
Whiteboard: [wptsync downstream][domsecurity-backlog] → [wptsync downstream]
| Assignee | ||
Comment 5•8 years ago
|
||
| Assignee | ||
Comment 6•8 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=61acf4e5c17026f14a67f68388a21ee0915c1be8
| Assignee | ||
Comment 7•8 years ago
|
||
Ran 30 tests and 161 subtests
OK : 27
PASS : 7
FAIL : 154
TIMEOUT: 3
New tests that have failures or other problems:
/trusted-types/Element-setAttributeNS.tentative.html
Element.setAttributeNS assigned via policy (successful HTML transformation): FAIL
Element.setAttributeNS assigned via policy (successful Script transformation): FAIL
Element.setAttributeNS assigned via policy (successful ScriptURL transformation): FAIL
Element.setAttributeNS assigned via policy (successful URL transformation): FAIL
/trusted-types/block-string-assignment-to-Element-setAttributeNS.tentative.html
Element.setAttributeNS assigned via policy (successful HTML transformation): FAIL
Element.setAttributeNS assigned via policy (successful Script transformation): FAIL
Element.setAttributeNS assigned via policy (successful ScriptURL transformation): FAIL
Element.setAttributeNS assigned via policy (successful URL transformation): FAIL
`Element.setAttributeNS = null` throws: FAIL
`Element.setAttributeNS = string` throws: FAIL
Updated•8 years ago
|
Whiteboard: [wptsync downstream] → [wptsync downstream][domsecurity-backlog]
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1b52d9026c67
[wpt PR 12969] - Element::setAttributeNS() modified to accept TrustedTypes, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/cea3f4e4d513
[wpt PR 12969] - Update wpt metadata, a=testonly
Comment 9•8 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/1b52d9026c67
https://hg.mozilla.org/mozilla-central/rev/cea3f4e4d513
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in
before you can comment on or make changes to this bug.
Description
•