Closed
Bug 1366734
Opened 7 years ago
Closed 5 years ago
Implement webkitURL legacy window alias for URL
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
DUPLICATE
of bug 1366738
People
(Reporter: zcorpan, Unassigned)
References
Details
Attachments
(1 file)
1.87 KB,
patch
|
annevk
:
review-
|
Details | Diff | Splinter Review |
See https://github.com/whatwg/url/issues/135 https://github.com/whatwg/url/pull/312 https://github.com/w3c/web-platform-tests/pull/6020 Also see https://github.com/heycam/webidl/issues/362 If there's any objection to implementing webkitURL in Gecko I suppose we can try to get it removed from WebKit/Chromium/EdgeHTML... If you're OK with this, please say so so that the spec change and test can be merged. :-)
Comment 1•7 years ago
|
||
I feel like Anne is best positioned to answer this but he's on vacation. Maybe baku has thoughts or can channel Anne's?
Flags: needinfo?(amarchesini)
Comment 2•7 years ago
|
||
Yes, we want to implement [LegacyWindowAlias]. Temporary solution would be to use NamedConstructor.
Flags: needinfo?(amarchesini)
Comment 3•7 years ago
|
||
I'll file a separate bug for [LegacyWindowAlias].
Assignee: nobody → amarchesini
Attachment #8872921 -
Flags: review?(annevk)
Comment 4•7 years ago
|
||
(In reply to Andrea Marchesini [:baku] from comment #3) > > I'll file a separate bug for [LegacyWindowAlias]. I think that's bug 1366738.
Updated•7 years ago
|
Priority: -- → P3
Comment 5•7 years ago
|
||
Comment on attachment 8872921 [details] [diff] [review] namedUrl.patch Review of attachment 8872921 [details] [diff] [review]: ----------------------------------------------------------------- I suggest we wait until we have LegacyWindowAlias since it's a) trivial and b) we might hit compat issues so it's better to have the right fix from the start.
Attachment #8872921 -
Flags: review?(annevk) → review-
Updated•7 years ago
|
Assignee: amarchesini → nobody
Assignee | ||
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
Comment 6•5 years ago
|
||
It seems that this was implemented in bug 1366738.
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•