Open Bug 1406337 Opened 7 years ago Updated 11 months ago

Remove comment handling from media type parser

Categories

(Core :: Networking, enhancement, P3)

Product:
Core
Component:
Networking
Type:
enhancement

Tracking

()

People

(Reporter: annevk, Unassigned)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: parity-chrome, parity-edge, parity-safari, Whiteboard: [necko-triaged]) 

> We include '(' in the trailing trim set to catch media-type comments,
> which are not at all standard, but may occur in rare cases.

Safari does not have this behavior and Edge does not have it for parameter values (and barely for media types). Let's remove it.

See https://github.com/whatwg/mimesniff/issues/33 for further analysis. (I'm planning to refine the media/MIME type parser in that standard to get browsers more closely aligned. Eventually there will be tests too, once I know what I think the details should be like. Feedback appreciated.)
It seems Chrome is willing to fix this per https://bugs.chromium.org/p/chromium/issues/detail?id=772343 though they'd appreciate some preliminary feedback from Firefox.
Flags: needinfo?(mcmanus)
Blocks: mimesniff
Based on the current information. Looks like we are not going to implement this soon, put it in our P3 bucket.
Priority: -- → P3
Whiteboard: [necko-triaged]
Depends on: 1423877
Severity: normal → S3

Clear a needinfo that is pending on an inactive user.

Inactive users most likely will not respond; if the missing information is essential and cannot be collected another way, the bug maybe should be closed as INCOMPLETE.

For more information, please visit BugBot documentation.

Flags: needinfo?(mcmanus)
You need to log in before you can comment on or make changes to this bug.