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)

enhancement

Tracking

()

RESOLVED INVALID

People

(Reporter: bkelly, Assigned: baku)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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.
Priority: -- → P3
Depends on: 1407540
Attached patch url.patchSplinter Review
This is not ready yet, it doesn't pass all the existing tests.
Assignee: nobody → amarchesini
Attachment #8922329 - Flags: feedback?(valentin.gosu)
Depends on: 1409405, 1410063, 1411932
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)
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.

Attachment

General

Created:
Updated:
Size: