Closed
Bug 1397151
Opened 7 years ago
Closed 7 years ago
Remove moz-chunked-text support from XHR
Categories
(Core :: DOM: Core & HTML, enhancement)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
mozilla58
| Tracking | Status | |
|---|---|---|
| firefox58 | --- | fixed |
People
(Reporter: baku, Assigned: baku)
References
Details
(Keywords: dev-doc-complete, site-compat)
Attachments
(1 file, 3 obsolete files)
|
15.73 KB,
patch
|
Details | Diff | Splinter Review |
In bug 1335365 I introduced Telemetry::MOZ_CHUNKED_TEXT_IN_XHR to count how often moz-chunked-text type is used in XHR. The result is: 0.16% I would like to proceed with removing this non-spec feature.
|
Assignee | |
Updated•7 years ago
|
Summary: Remove moz-blob support from XHR → Remove moz-chunked-text support from XHR
|
|
Assignee | |
Comment 1•7 years ago
|
||
Assignee: nobody → amarchesini
Attachment #8904885 -
Flags: review?(bugs)
|
|
Assignee | |
Comment 2•7 years ago
|
||
Attachment #8904885 -
Attachment is obsolete: true
Attachment #8904885 -
Flags: review?(bugs)
Attachment #8904886 -
Flags: review?(bugs)
|
||
Comment 3•7 years ago
|
||
Comment on attachment 8904886 [details] [diff] [review]
moz_text.patch
Is domerr.msg ever localized? If it is, could you ask some l10n folks to approve this. (But I think it isn't localized).
And if we want to be super conservative for FF57, this could perhaps land after the branch date.
Attachment #8904886 -
Flags: review?(bugs) → review+
|
|
Assignee | |
Comment 4•7 years ago
|
||
Green on try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=83377f20f8b02d994f47a56bc2ceb7562857f266&selectedJob=128876770
Please, land this patch when 57 is the new beta.
Keywords: checkin-needed
Whiteboard: to land when 57 is in beta
|
|
Assignee | |
Updated•7 years ago
|
Keywords: dev-doc-needed
|
|
Assignee | |
Comment 5•7 years ago
|
||
Attachment #8904886 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 6•7 years ago
|
||
Attachment #8905107 -
Attachment is obsolete: true
|
|
Assignee | |
Comment 7•7 years ago
|
||
NI RyanVM to land this patch after 57.
Flags: needinfo?(ryanvm)
Keywords: checkin-needed
Pushed by amarchesini@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9fb93fcf2b8e
Remove the support for moz-chunk-text type in XHR, r=smaug
|
||
Comment 10•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox58:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58
|
||
Comment 11•7 years ago
|
||
Looks like this is related to bug 1120171, so marking it as such for posterity.
Blocks: 1120171
|
||
Comment 12•7 years ago
|
||
Posted the site compatibility note: https://www.fxsitecompat.com/en-CA/docs/2017/prefixed-xmlhttprequest-response-types-including-moz-blob-are-no-longer-supported/
Keywords: site-compat
|
||
Comment 13•7 years ago
|
||
I have documented this, by adding a note to the responseType page and removing the information about those values:
https://developer.mozilla.org/en-US/docs/Web/API/XMLHttpRequest/responseType#Browser_compatibility
And by adding a note to the Fx58 rel notes:
https://developer.mozilla.org/en-US/Firefox/Releases/58#APIs_2
Let me know if that looks OK. Thanks!
Keywords: dev-doc-needed → dev-doc-complete
|
||
Updated•6 years ago
|
Component: DOM → DOM: Core & HTML
You need to log in
before you can comment on or make changes to this bug.
Description
•