Summary: | [Qt] plugin is loaded to the web process via MainResourceLoader::substituteMIMETypeFromPluginDatabase | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Balazs Kelemen <kbalazs> | ||||
Component: | Plug-ins | Assignee: | Balazs Kelemen <kbalazs> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | japhet, jochen, kenneth, webkit.review.bot, yael | ||||
Priority: | P2 | Keywords: | Qt, QtTriaged | ||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Balazs Kelemen
2012-05-15 09:00:21 PDT
This code has been added in https://bugs.webkit.org/show_bug.cgi?id=32572. It's possible to reimplement it in a way that fits with WebKit2 by going trough PluginData -> WebPlatformStrategies -> UI process but I have trouble with testing this functionality. I could not load an swf file with QtTestBrowser on unpatched trunk - actually I found an swf file that loads but it doesn't show anything else than black. Do you think it should work on trunk? Could you give an example I can test with? Or maybe we should just drop that? Created attachment 146053 [details]
Patch
Comment on attachment 146053 [details]
Patch
I very much agree with this, the symptom (webprocess shouldn't load plugins through wk1 code path by use of PluginDatabase) as well as the removal of this hack :)
Comment on attachment 146053 [details] Patch Clearing flags on attachment: 146053 Committed r121423: <http://trac.webkit.org/changeset/121423> All reviewed patches have been landed. Closing bug. |