Closed Bug 1504344 Opened 6 years ago Closed 5 years ago

Remove the pref dom.xhr.lowercase_header.enabled

Categories

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

Product:
Core
Component:
DOM: Core & HTML
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla70
Tracking Flags:
Tracking Status
firefox65 --- wontfix
firefox70 --- fixed

People

(Reporter: twisniewski, Assigned: twisniewski)

References

Details

Attachments

(1 file)

Bug 1504344 - Remove the pref dom.xhr.lowercase_header.enabled; r?baku
(deleted), text/x-phabricator-request
Details 
The pref was flipped to true in bug 1398718. It should be safe to remove it outright if there is still no fallout after a release cycle or two.
Priority: -- → P3
Component: DOM → DOM: Core & HTML

Andrew, Henri, do you think it's been long enough for us to remove this pref? It landed in 64, and I haven't seen any fallout yet that is actionable. (Removing it would also help simplify the patch for bug 1491010, where I plan on having the XHR code re-use the same code Fetch is using, which doesn't have this special-case).

Flags: needinfo?(overholt)
Flags: needinfo?(hsivonen)

If we've shipped in this state since 64 without actionable problems, removing configurability seems OK to me.

Flags: needinfo?(hsivonen)

My only concern is that we haven't had an ESR cycle since shipping it but I defer to Henri so let's get rid of it :)

Flags: needinfo?(overholt)

Remove the pref dom.xhr.lowercase_header.enabled, as we are unaware of any actionable compat concerns now that bug 1540688 landed, and an ESR had been spun off.

Pushed by twisniewski@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/501b7b7bab15
Remove the pref dom.xhr.lowercase_header.enabled; r=baku

https://hg.mozilla.org/mozilla-central/rev/501b7b7bab15

Status: NEW → RESOLVED
Closed: 5 years ago
status-firefox70: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla70
Assignee: nobody → twisniewski
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: