Closed Bug 1572349 Opened 2 months ago Closed Last month

Add a document URL resolver method

Categories

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

task

Tracking

()

RESOLVED FIXED
mozilla70
Tracking Status
firefox70 --- fixed

People

(Reporter: saschanaz, Assigned: saschanaz, Mentored)

Details

Attachments

(1 file)

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.

Assignee: nobody → saschanaz
Status: NEW → ASSIGNED
Priority: -- → P3
Mentor: mcaceres

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

Keywords: checkin-needed

Pushed by nbeleuzu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8e9d1223d391
Add Document::ResolveWithBaseURI r=bzbarsky

Keywords: checkin-needed
Flags: needinfo?(saschanaz)
Keywords: checkin-needed

Pushed by ncsoregi@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bd6df046c082
Add Document::ResolveWithBaseURI. r=bzbarsky

Keywords: checkin-needed
Status: ASSIGNED → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
You need to log in before you can comment on or make changes to this bug.