Closed Bug 1161625 Opened 5 years ago Closed 5 years ago

Use nsIURI directly, not URL, for base URI bits in fetch Response/Request

Categories

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

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: bzbarsky, Assigned: bzbarsky)

Details

Attachments

(1 file)

There's no need for the extra parsing/serialization we end up doing right now.

Also, while I was here I fixed things to use the entry document's base URI instead of the callee document's document URI.
Summary: Use nsIURI directly, not URL for base URI bits in fetch Response/Request → Use nsIURI directly, not URL, for base URI bits in fetch Response/Request
Nikhil, is there a particular place where tests for this stuff live?  We should really add that test for entry global base URI...
Flags: needinfo?(nsm.nikhil)
dom/tests/mochitests/fetch
Flags: needinfo?(nsm.nikhil)
No web platform tests for this stuff?
Flags: needinfo?(nsm.nikhil)
back then we couldn't run wpt tests on things hidden behind a pref. not sure if that is still the case. It should be fairly trivial to port most of the tests to wpt.
Flags: needinfo?(nsm.nikhil)
https://hg.mozilla.org/mozilla-central/rev/34fae3ffbf62
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.