1572349 - Add a document URL resolver method

Status: RESOLVED FIXED

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

Description

[Kagami Rosylight [:saschanaz] (they/them) (OOO until 2024-10-06)]

Discussion on dev-platform: https://groups.google.com/forum/#!msg/mozilla.dev.platform/y_eEf14fLu4/ECG_Mq7hAQAJ

We have several cases that needs to resolve urls with document.baseURI, so adding proper helper function will be helpful.

One function is being implemented in https://phabricator.services.mozilla.com/D41047 where it receives nsAString and returns nsIURI, and maybe another new overload that receives an nsIURI would be helpful too.

Comment 1

[Kagami Rosylight [:saschanaz] (they/them) (OOO until 2024-10-06)]

Attachment: Bug 1572349 - Add Document::ResolveWithBaseURI

This adds a helper method that enables simpler code in D41047 and D36714.

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8e9d1223d391
Add Document::ResolveWithBaseURI r=bzbarsky

Comment 3

[Noemi Erli[:noemi_erli]]

Backed out changeset 8e9d1223d391 (Bug 1572349) for causing build bustages in Document.cpp CLOSED TREE

Push with failure: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=8e9d1223d391da60c16ea3e4087123f6c3b9f3a4

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

Backout: https://hg.mozilla.org/integration/autoland/rev/8b8878633443587a44118fad8f247c6b9cd6adb9

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bd6df046c082
Add Document::ResolveWithBaseURI. r=bzbarsky

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/bd6df046c082