Chromium Code Reviews Chromium Code Reviews
Issue
18364005:
Don't override application/octet-stream MIME type. (Closed)
DescriptionDon't override application/octet-stream MIME type.
When enumerating candidate plug-ins for handling a document,
PluginList::GetPluginInfoArray() attemps to match the MIME type of the
document, and then matches the file type based on the URL. Matching by
file type is done if the MIME type of the document is either empty or if
is application/octet-stream.
This change disallows plugin matching based on file type if the MIME
type is application/octet-stream. This will, for example, prevent
http://example.com/foo.pdf from being associated with the PDF plug-in if
it is served with a MIME type of application/octet-stream.
As a side-effect, this brings the BufferedResourceHandler's decision of
whether a resource should be rendered or downloaded closer in line with
Blink's.
BUG=104331
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213119
Patch Set 1 : #Patch Set 2 : Merge with r212359 #
Total comments: 2
Patch Set 3 : Address comment. #Patch Set 4 : Merge with r212605 to catch up with file move. #Patch Set 5 : Merge with r212882 to catch up with namespace changes. #
Messages
Total messages: 12 (0 generated)
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||
