Closed
Bug 1488349
Opened 6 years ago
Closed 6 years ago
Remove non-HTTP parts of nsMIMEHeaderParamImpl from m-c
Categories
(Core :: Networking: HTTP, enhancement, P5)
Core
Networking: HTTP
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: hsivonen, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [overhead:noted][necko-triaged])
It appears that nsIMIMEHeaderParam is used from JS in m-c only in unit tests. Also, most of the interface is unused in m-c, since m-c only uses the HTTP part.
nsMIMEHeaderParamImpl.cpp should be deCOMtaminated and the non-HTTP parts removed from m-c. (A quick look suggests that no processing differs depending on HTTP_FIELD_ENCODING or MIME_FIELD_ENCODING.)
Updated•6 years ago
|
Whiteboard: [overhead:noted]
Comment 1•6 years ago
|
||
ni Jason to find an owner and set the priority.
Flags: needinfo?(jduell.mcbugs)
Comment 2•6 years ago
|
||
Hello Henri,
I see the clients in JS of nsIMIMEHeaderParam are devtool[1]/toolkit[2] instead of unit tests.
Hence, it's hard to deCOMtaminated with these clients stand.
You're correct, lots of API in nsIMIMEHeaderParam.idl should not exposed.
However, if we couldn't deCOMtaminated it, it's not very attractive to shrink the list of API.
What do you think?
[1] https://searchfox.org/mozilla-central/rev/807a37c670c093b6e5201841a7c5315ba67ba8d5/devtools/startup/devtools-startup.js#990
[2] https://searchfox.org/mozilla-central/rev/924e3d96d81a40d2f0eec1db5f74fc6594337128/toolkit/content/contentAreaUtils.js#593
Flags: needinfo?(hsivonen)
Reporter | ||
Comment 3•6 years ago
|
||
(In reply to Junior Hsu from comment #2)
> Hello Henri,
> I see the clients in JS of nsIMIMEHeaderParam are devtool[1]/toolkit[2]
> instead of unit tests.
> Hence, it's hard to deCOMtaminated with these clients stand.
>
> You're correct, lots of API in nsIMIMEHeaderParam.idl should not exposed.
> However, if we couldn't deCOMtaminated it, it's not very attractive to
> shrink the list of API.
>
> What do you think?
Good point. I missed the devtools callers. Yeah, deCOMtamination doesn't work then.
Still, it would make sense to check how much browser-irrelevant email-related dead-in-Firefox code there is.
Flags: needinfo?(hsivonen)
Summary: DeCOMtaminate nsMIMEHeaderParamImpl and remove the non-HTTP parts from m-c → Non-HTTP parts of nsMIMEHeaderParamImpl from m-c
Updated•6 years ago
|
Flags: needinfo?(jduell.mcbugs)
Priority: -- → P5
Whiteboard: [overhead:noted] → [overhead:noted][necko-triaged]
Reporter | ||
Updated•6 years ago
|
Summary: Non-HTTP parts of nsMIMEHeaderParamImpl from m-c → Remove non-HTTP parts of nsMIMEHeaderParamImpl from m-c
Comment 4•6 years ago
|
||
I did some archeology.
We split the parser for MIME(TB and dev tool) and http header in bug 601933.
However, it breaks lots of sites in the field though it satisfies the spec.
We disable the effect in bug 858337, which meets the Comment 0.
Hence, splitting two functions with same functionality help it easy to revert.
A stale result [1] shows still considerable breakage.
[1] https://bugs.chromium.org/p/chromium/issues/detail?id=66694#c27
We might figure we can fix bug 601933 one day.
Let's rather leave the code unchanged.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•