Closed
Bug 1406933
Opened 7 years ago
Closed 6 years ago
Make URL() use MozURL so it can be identical on main thread and workers
Categories
(Core :: DOM: Workers, enhancement, P3)
Core
DOM: Workers
Tracking
()
RESOLVED
INVALID
People
(Reporter: bkelly, Assigned: baku)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
61.34 KB,
patch
|
Details | Diff | Splinter Review |
Now that bug 1394906 has landed we should re-write URL() in terms of MozURL. This will let us unify the code for main thread and workers. It will also avoid a bunch of perf-sucking runnables in the worker code.
Updated•7 years ago
|
Priority: -- → P3
Assignee | ||
Comment 1•7 years ago
|
||
This is not ready yet, it doesn't pass all the existing tests.
Assignee: nobody → amarchesini
Attachment #8922329 -
Flags: feedback?(valentin.gosu)
Assignee | ||
Updated•7 years ago
|
Comment on attachment 8922329 [details] [diff] [review] url.patch I'll delay this until the threadsafe URI work is done.
Attachment #8922329 -
Flags: feedback?(valentin.gosu)
Assignee | ||
Comment 3•6 years ago
|
||
This bug is not needed anymore. URL Worker and URL Main-thread implementations have been merged in bug 1454656
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•