Closed
Bug 1161625
Opened 9 years ago
Closed 9 years ago
Use nsIURI directly, not URL, for base URI bits in fetch Response/Request
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
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.
Assignee | ||
Updated•9 years ago
|
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
Assignee | ||
Comment 1•9 years ago
|
||
We seem to have no tests in our tree for the entry settings base URI bit. :(
Attachment #8601670 -
Flags: review?(nsm.nikhil)
Attachment #8601670 -
Flags: review?(nsm.nikhil) → review+
Assignee | ||
Comment 3•9 years ago
|
||
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)
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: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
Updated•5 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•