Closed Bug 1499136 Opened Last year Closed Last year

Disable dom.xhr.standard_content_type_normalization for now due to webcompat issues.

Categories

(Core :: DOM: Core & HTML, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla64
Tracking Status
firefox-esr60 --- unaffected
firefox62 --- unaffected
firefox63 --- unaffected
firefox64 + fixed

People

(Reporter: twisniewski, Assigned: twisniewski)

References

Details

(Keywords: site-compat)

Attachments

(1 file)

Given that we've had three reports against bug 1454325 indicating that the spec isn't quite web-compatible, Anne and I have decided to disable to disable the related about:config flag for now: dom.xhr.standard_content_type_normalization
Priority: -- → P1
Assignee: nobody → twisniewski
Disable dom.xhr.standard_content_type_normalization for now due to webcompat issues
Alright, I don't see anything worrying on the try-run, so I've submitted the pref-flipping patch for review.
Henri, do you think you'll have time to review this soon? I'd like to get this in before beta splits off if possible, to prevent the need for an uplift.
Flags: needinfo?(hsivonen)
(In reply to Thomas Wisniewski [:twisniewski] from comment #4)
> Henri, do you think you'll have time to review this soon? I'd like to get
> this in before beta splits off if possible, to prevent the need for an
> uplift.

I reviewed it earlier already and Phabricator shows it as reviewed to me. If Phabricator shows you something different and landing needs to bypass Phabricator to make progress, please consider this as an r+.
Flags: needinfo?(hsivonen)
Ah, knowing me I just missed that it had been given an r+ (I'm still not quite used to Phabricator). Thanks!

Requesting check-in.
Keywords: checkin-needed
Pushed by ccoroiu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/affe4a683227
Disable dom.xhr.standard_content_type_normalization for now due to webcompat issues; r=hsivonen
Keywords: checkin-needed
Backout by dluca@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/7aafde5f425e
Backed out changeset affe4a683227 for mochitest failures in dom/xhr/tests/test_XHRSendData.htm. CLOSED TREE
Backed out changeset affe4a683227 (bug 1499136) for mochitest failures in dom/xhr/tests/test_XHRSendData.htm. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#?job_id=206214083&repo=autoland&lineNumber=4226

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=affe4a683227c556ec3d21bbf9e9024fa4a71a4a
Flags: needinfo?(twisniewski)
This is actually a race in the test which I didn't realize could happen when I altered it in bug 1454325 (I'm surprised that I had the misfortune of not hitting this in my own try-runs).

I've updated the patch here to avoid that race, by having it do the required setup work in a non-racy sequence during onload, before the createFiles function tries to run the tests.

As I don't see any other weird signs in that try-run, and this is just a minor change to that test (which I verified fixes the glitch for me locally), I'm going to carry over the r+ and re-request check-in.
Flags: needinfo?(twisniewski)
Keywords: checkin-needed
Pushed by ebalazs@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0bb500ba0c07
Disable dom.xhr.standard_content_type_normalization for now due to webcompat issues; r=hsivonen
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/0bb500ba0c07
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
https://hg.mozilla.org/projects/larch/rev/7aafde5f425eaebe15aee67f05cb7c6ad02b9324
Backed out changeset affe4a683227 (bug 1499136) for mochitest failures in dom/xhr/tests/test_XHRSendData.htm. CLOSED TREE

https://hg.mozilla.org/projects/larch/rev/0bb500ba0c07c122610452184a162166ff47ce96
Bug 1499136 - Disable dom.xhr.standard_content_type_normalization for now due to webcompat issues; r=hsivonen
Byron, I'm confused. Was this patch backed out, and then pushed again, or was it just backed out?

If it was just backed out, could I see a treeherder link or whatever inspired the backout?
Flags: needinfo?(docfaraday)
So all I did was merge m-c to larch for a project that the necko/webrtc teams are working on, and one of our standard hg extensions went off the rails and spammed bugzilla pretty badly. It seems to have tried to leave comments on every bug that was merged into larch, before hitting an error and stopping.
Flags: needinfo?(docfaraday)
OK, thanks.
Component: DOM → DOM: Core & HTML
You need to log in before you can comment on or make changes to this bug.