RESOLVED FIXED 60546
[GTK] Implement Plugin Process 
https://bugs.webkit.org/show_bug.cgi?id=60546
Summary [GTK] Implement Plugin Process
Carlos Garcia Campos
Reported 2011-05-10 06:05:40 PDT
GTK port doesn't support plugin process yet. The main advantage of using another process for plugins for the GTK port is that we can use the flash plugin (that uses gtk2) even if webkit is built with gtk3.
Attachments
Patch (19.30 KB, patch)
2011-06-07 06:25 PDT, Carlos Garcia Campos 		no flags
DetailsFormatted DiffDiff
Updated patch (19.67 KB, patch)
2011-06-20 05:08 PDT, Carlos Garcia Campos 		mrobinson: review+
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Carlos Garcia Campos
Comment 1 2011-06-07 06:25:27 PDT
Created attachment 96235 [details] Patch
WebKit Review Bot
Comment 2 2011-06-07 06:28:36 PDT
Attachment 96235 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'Source/WebKit2/ChangeLog', u..." exit_code: 1 Source/WebKit2/UIProcess/Launcher/gtk/ProcessLauncherGtk.cpp:72: Use 0 instead of NULL. [readability/null] [5] Total errors found: 1 in 8 files If any of these errors are false positives, please file a bug against check-webkit-style.
Carlos Garcia Campos
Comment 3 2011-06-20 05:08:04 PDT
Created attachment 97780 [details] Updated patch Patch updated to apply on current git master, also simplified te build system a bit since webkit2 doesn't support gtk2 anymore.
WebKit Review Bot
Comment 4 2011-06-20 05:11:40 PDT
Attachment 97780 [details] did not pass style-queue: Failed to run "['Tools/Scripts/check-webkit-style', '--diff-files', u'ChangeLog', u'Source/WebKit2/ChangeLog', u..." exit_code: 1 Source/WebKit2/UIProcess/Launcher/gtk/ProcessLauncherGtk.cpp:72: Use 0 instead of NULL. [readability/null] [5] Total errors found: 1 in 9 files If any of these errors are false positives, please file a bug against check-webkit-style.
Martin Robinson
Comment 5 2011-06-20 09:39:19 PDT
Comment on attachment 97780 [details] Updated patch Okay! Let's do this.
Carlos Garcia Campos
Comment 6 2011-06-20 11:06:24 PDT
This still depends on bug #60629, so I'll land this as soon as it's fixed. Thanks for reviewing.
Carlos Garcia Campos
Comment 7 2011-07-14 10:50:06 PDT
Committed r91015: <http://trac.webkit.org/changeset/91015>
Note You need to log in before you can comment on or make changes to this bug.