1558923 - Simplify URL API

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task)

Description

[Andrea Marchesini [:baku]]

nsIURI is thread-safe. We don't need to have 2 different implementations for URL API, one for workers and one for main-thread.

Comment 1

[Andrea Marchesini [:baku]]

Attachment: Bug 1558923 - Simplify URL API, r=valentin

Comment 2

[Andrea Marchesini [:baku]]

Attachment: Bug 1558923 - Remove nsContentUtils::GetThreadSafe{ASCII,UTF8}Origin, r?valentin

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d0411a373ace
Simplify URL API, r=valentin
https://hg.mozilla.org/integration/autoland/rev/0bb5857422f2
Remove nsContentUtils::GetThreadSafe{ASCII,UTF8}Origin, r=valentin

Comment 4

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/87d2acfb0ab3
Backed out 2 changesets for build bustages on dom/URL.h. CLOSED TREE

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d8a688f1b7ce
Simplify URL API, r=valentin
https://hg.mozilla.org/integration/autoland/rev/12c977801cb2
Remove nsContentUtils::GetThreadSafe{ASCII,UTF8}Origin, r=valentin

Comment 6

[Noemi Erli[:noemi_erli]]

Backed out 2 changesets (Bug 1558923) for failures in nsContentUtils.cpp

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=7ba9d3d60a935bb732cb464772922c012f961490

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=251681053&repo=autoland&lineNumber=1909

Backout: https://hg.mozilla.org/integration/autoland/rev/b678bad258d44b049b8a2a916431432ea823e36d

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f758b5ccd0c0
Simplify URL API, r=valentin
https://hg.mozilla.org/integration/autoland/rev/643de99320a8
Remove nsContentUtils::GetThreadSafe{ASCII,UTF8}Origin, r=valentin

Comment 8

[Narcis Beleuzu [:NarcisB]]

Backed out for mochitest crashes on AssertIsOnMainThread()

Backout link: https://hg.mozilla.org/integration/autoland/rev/fcdbd2fce46323f30ca026bbb14bd32e1f6538b2
Push link: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Crunnable&selectedJob=251699658&revision=643de99320a862b7a68a3e2baedd3395699a9e17
Log link: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=251699658&repo=autoland&lineNumber=1840

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/dd5de4428cf9
Simplify URL API, r=valentin
https://hg.mozilla.org/integration/autoland/rev/622449608100
Remove nsContentUtils::GetThreadSafe{ASCII,UTF8}Origin, r=valentin

Comment 10

[Stefan Hindli [:stefan_hindli]]

https://hg.mozilla.org/mozilla-central/rev/dd5de4428cf9
https://hg.mozilla.org/mozilla-central/rev/622449608100