Closed Bug 1838013 Opened 1 years ago Closed 1 year ago

XHR should throw a "SyntaxError" DOMException

Categories

(Core :: DOM: Networking, defect, P2)

defect

Tracking

()

RESOLVED FIXED
119 Branch
Tracking Status
firefox119 --- fixed

People

(Reporter: edgul, Assigned: edgul)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

In WPTs url/failure.html, on some inputs we are failing XHR, but passing URL's href. The issue is likely that we are returning the incorrect error from the relevant cpp. Let's fix this so we are consistent for each input of our URL tests.

This appears to be similar to Bug 1817591

Severity: -- → S3
Priority: -- → P2
See Also: → 1817591
Whiteboard: [necko-triaged]
Assignee: nobody → edgul
Status: NEW → ASSIGNED

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:edgul, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(valentin.gosu)
Flags: needinfo?(edgul)

This will be landed with Bug 1723456. Coming soon.

Flags: needinfo?(valentin.gosu)
Flags: needinfo?(edgul)
Pushed by eguloien@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d34f90c04def Make XHR URL failure throw syntax error DOMException to pass failure.html WPTs. r=valentin
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 119 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: