Bug 118713 - [WK2] Support findPlugin through file extension when a mimetype is not null
Summary: [WK2] Support findPlugin through file extension when a mimetype is not null
Status: RESOLVED WONTFIX
Alias: None
Product: WebKit
Classification: Unclassified
Component: Plug-ins (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P4 Trivial
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-07-15 22:32 PDT by Sangho Kim
Modified: 2013-09-13 09:02 PDT (History)
5 users (show)

See Also:


Attachments
patch (1.79 KB, patch)
2013-07-15 22:38 PDT, Sangho Kim
andersca: review-
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion (553.16 KB, application/zip)
2013-07-15 23:44 PDT, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Sangho Kim 2013-07-15 22:32:52 PDT
I think it's good for users if users mistake to write invalid mimetypes, WebKit finds plugin through file extensions.
Now, WebKit finds plugin through file extensions when a mimetype is only null.
How do you think about that?
Comment 1 Sangho Kim 2013-07-15 22:38:50 PDT
Created attachment 206724 [details]
patch

change the logic.
Comment 2 Build Bot 2013-07-15 23:44:32 PDT
Comment on attachment 206724 [details]
patch

Attachment 206724 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/1089099

New failing tests:
fullscreen/full-screen-iframe-with-max-width-height.html
Comment 3 Build Bot 2013-07-15 23:44:33 PDT
Created attachment 206728 [details]
Archive of layout-test-results from webkit-ews-06 for mac-mountainlion

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: webkit-ews-06  Port: mac-mountainlion  Platform: Mac OS X 10.8.3
Comment 4 Sangho Kim 2013-07-15 23:54:36 PDT
I don't understand why rayout test fail...
I just changed findplugin code a little...
Comment 5 Philippe Normand 2013-08-08 08:32:36 PDT
Comment on attachment 206724 [details]
patch

View in context: https://bugs.webkit.org/attachment.cgi?id=206724&action=review

> Source/WebKit2/ChangeLog:3
> +        Support findPlugin through file extentions when a mimetype isn't null.

Needs [WK2] prefix. I will retitle the bug.

> Source/WebKit2/ChangeLog:8
> +        To find plugin through file extentions even if mimetype isn't null.

It'd be good to elaborate a bit here. Why is this change needed?
Comment 6 Alexey Proskuryakov 2013-08-08 10:04:57 PDT
It’s generally not a good idea to disregard an explicitly provided MIME type.
Comment 7 Anders Carlsson 2013-09-13 09:02:17 PDT
(In reply to comment #6)
> It’s generally not a good idea to disregard an explicitly provided MIME type.

I agree, but I don't feel too strongly about it.

However, if we don't have any evidence of real world web sites that are fixed by this change I don't think we should make it.