Closed Bug 1366738 Opened 4 years ago Closed 2 years ago

Implement [LegacyWindowAlias] extended attribute (for WebKitCSSMatrix, webkitURL)


(Core :: DOM: Bindings (WebIDL), enhancement, P3)




Tracking Status
firefox69 --- fixed


(Reporter: zcorpan, Assigned: saschanaz, NeedInfo)



(Keywords: dev-doc-needed)


(1 file)

Blocks: 1366734
Should we do this?
Flags: needinfo?(annevk)
Flags: needinfo?(amarchesini)
Yes, we should but it requires an important change in the WebIDL codegen.
A temporary solution would be use NamedConstructor as discussed here:
Flags: needinfo?(amarchesini)
Andrea and I discussed this and it doesn't seem important to tackle this before Firefox 57, especially as bz said it would be non-trivial.
Flags: needinfo?(annevk)
Priority: -- → P3
Component: DOM → DOM: Core & HTML
Component: DOM: Core & HTML → DOM: Bindings (WebIDL)
Assignee: nobody → saschanaz

Add [LegacyWindowAlias] extended attribute support for WebKitCSSMatrix and webkitURL.

Depends on: 1397945
Blocks: 1170774
Depends on: 1560480
Keywords: checkin-needed

Pushed by
Implement [LegacyWindowAlias] r=bzbarsky

Keywords: checkin-needed

Backed out changeset 509af1e0771d for causing failures in test_WebKitCSSMatrix.html

Backout link:

Push with failures:

Failure log:

[task 2019-07-03T07:10:30.318Z] 07:10:30 INFO - TEST-UNEXPECTED-FAIL | dom/tests/mochitest/general/test_WebKitCSSMatrix.html | Test that inverting an invertible matrix throws. - Test that inverting an invertible matrix throws.: assert_throws: Inverting an invertible matrix should throw. function "function() { m.inverse(); }" did not throw
[task 2019-07-03T07:10:30.318Z] 07:10:30 INFO - TEST-PASS | dom/tests/mochitest/general/test_WebKitCSSMatrix.html | Test for WebKitCSSMatrix - Test for WebKitCSSMatrix: Elided 4 passes or known failures.

Flags: needinfo?(saschanaz)
Flags: needinfo?(saschanaz)
Keywords: checkin-needed

Pushed by
Implement [LegacyWindowAlias] r=bzbarsky

Keywords: checkin-needed
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

After review, I do not believe anything here reaches the level of needing to be documented on MDN. If anyone can think of a reason that wouldn't be true, let me know.

Keywords: dev-doc-needed

I think this should redirect to and this should say that SVGMatrix and WebKitCSSMatrix are legacy aliases to DOMMatrix in a Window context. (Though SVGMatrix is not yet implemented as an alias.) Recommend using DOMMatrix instead of WebKitCSSMatrix. could say that webkitURL exists as a legacy alias in a Window context. Recommend using URL instead.

Flags: needinfo?(eshepherd)
Duplicate of this bug: 1366734

Adding back dev-doc-needed for my above comment

Keywords: dev-doc-needed
You need to log in before you can comment on or make changes to this bug.