Bug 110656 - pluginLoadStrategy should take a WKDictionaryRef for extensibility
Summary: pluginLoadStrategy should take a WKDictionaryRef for extensibility
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Anders Carlsson
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-22 16:03 PST by Anders Carlsson
Modified: 2013-02-22 16:45 PST (History)
1 user (show)

See Also:


Attachments
Patch (16.90 KB, patch)
2013-02-22 16:06 PST, Anders Carlsson
jberlin: review+
webkit-ews: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Anders Carlsson 2013-02-22 16:03:10 PST
pluginLoadStrategy should take a WKDictionaryRef for extensibility
Comment 1 Anders Carlsson 2013-02-22 16:06:00 PST
Created attachment 189856 [details]
Patch
Comment 2 Early Warning System Bot 2013-02-22 16:19:37 PST
Comment on attachment 189856 [details]
Patch

Attachment 189856 [details] did not pass qt-wk2-ews (qt):
Output: http://queues.webkit.org/results/16722168
Comment 3 Jessie Berlin 2013-02-22 16:40:17 PST
Comment on attachment 189856 [details]
Patch

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

r=me with the Qt build fix.

> Source/WebKit2/UIProcess/WebPageProxy.messages.in:316
> +    GetPluginPath(WTF::String mimeType, WTF::String urlString, WTF::String frameURLString, WTF::String pageURLString) -> (WTF::String pluginPath, uint32_t pluginLoadPolicy)

Not sure if this change in indentation was intentional?
Comment 4 Anders Carlsson 2013-02-22 16:45:19 PST
Committed r143815: <http://trac.webkit.org/changeset/143815>