Bug 105564 - [GTK] Remove plugin process configuration option
Summary: [GTK] Remove plugin process configuration option
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Martin Robinson
URL:
Keywords:
Depends on:
Blocks: 87126
  Show dependency treegraph
 
Reported: 2012-12-20 12:10 PST by Martin Robinson
Modified: 2012-12-23 12:23 PST (History)
2 users (show)

See Also:


Attachments
Patch (8.59 KB, patch)
2012-12-20 12:33 PST, Martin Robinson
cgarcia: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Martin Robinson 2012-12-20 12:10:44 PST
Instead of allowing building without the WebKit2 plugin process, it should always be built for the GTK+ port.
Comment 1 Martin Robinson 2012-12-20 12:33:53 PST
Created attachment 180387 [details]
Patch
Comment 2 Carlos Garcia Campos 2012-12-20 23:55:32 PST
Comment on attachment 180387 [details]
Patch

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

This will automatically fix bug #89451. Thanks!

> Source/WebKit2/ChangeLog:13
> +

I think in-process plugin process is indeed incompatible in GTK+, because WebKit2 depends on GTK+ 3 and plugin process requires GTK+ 2 to work. When built in-process it would use GTK+ 3 and some plugins like flash won't work. You might want to mention it in the ChangeLog to make it clear we are not removing it just for the sake of reducing the configuration options.
Comment 3 Carlos Garcia Campos 2012-12-21 00:04:42 PST
Comment on attachment 180387 [details]
Patch

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

Something happened in my browser and last comment was not included in the comment

> configure.ac:1089
>     PKG_CHECK_MODULES(GTK2, gtk+-2.0 >= $GTK2_REQUIRED_VERSION gail >= $GAIL2_REQUIRED_VERSION)

Instead of checking again here if we are building webkit2, you could just move this above to the previous if test "$enable_webkit2" = "yes" block and check if gtk2 is present before unix printing.
Comment 4 Martin Robinson 2012-12-23 12:23:24 PST
Committed r138425: <http://trac.webkit.org/changeset/138425>