Closed
Bug 1366734
Opened 8 years ago
Closed 6 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•8 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•8 years ago
|
||
Yes, we want to implement [LegacyWindowAlias]. Temporary solution would be to use NamedConstructor.
Flags: needinfo?(amarchesini)
Comment 3•8 years ago
|
||
I'll file a separate bug for [LegacyWindowAlias].
Assignee: nobody → amarchesini
Attachment #8872921 -
Flags: review?(annevk)
Comment 4•8 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•8 years ago
|
Priority: -- → P3
Comment 5•8 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•8 years ago
|
Assignee: amarchesini → nobody
Assignee | ||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
Comment 6•6 years ago
|
||
It seems that this was implemented in bug 1366738.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•