Bug 89451

Summary:

[WK2][X11] Fix build break when building webkit2 without plugin process.

Product:

WebKit

Reporter:

Mariusz Grzegorczyk <mariusz.g>

Component:

WebKit2

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED INVALID

Severity:

Normal

CC:

andersca, cgarcia, kbalazs, mrobinson

Priority:

Version:

528+ (Nightly build)

Hardware:

Unspecified

OS:

Unspecified

Attachments:

Description	Flags
NetscapePluginModuleX11 fails to compile	none
patch	none
updated with Balazs' comments	andersca: review-

Mariusz Grzegorczyk

Reported 2012-06-19 02:48:29 PDT

When trying to build WK2 by running: ./Tools/Scripts/build-webkit --gtk --enable-plugin-process=no ,build break occurs: ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp: In member function ‘bool WebKit::NetscapePluginModule::getPluginInfoForLoadedPlugin(WebKit::RawPluginMetaData&)’: ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:115:17: error: invalid use of incomplete type ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/NetscapePluginModule.h:40:7: error: forward declaration of ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:119:17: error: invalid use of incomplete type ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/NetscapePluginModule.h:40:7: error: forward declaration of ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:125:13: error: invalid use of incomplete type ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/NetscapePluginModule.h:40:7: error: forward declaration of ‘struct WebKit::RawPluginMetaData’ ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp: In static member function ‘static bool WebKit::NetscapePluginModule::getPluginInfo(const WTF::String&, WebKit::PluginModuleInfo&)’: ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:132:23: error: aggregate ‘WebKit::RawPluginMetaData metaData’ has incomplete type and cannot be defined ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:133:10: error: ‘PluginProcessProxy’ has not been declared ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp: In static member function ‘static bool WebKit::NetscapePluginModule::scanPlugin(const WTF::String&)’: ../../Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:185:23: error: aggregate ‘WebKit::RawPluginMetaData metaData’ has incomplete type and cannot be defined

Attachments
NetscapePluginModuleX11 fails to compile (14.79 KB, text/plain) 2012-11-26 14:15 PST, Jonas Heinrich	no flags	Details
patch (4.72 KB, patch) 2013-03-01 03:20 PST, Mariusz Grzegorczyk	no flags	Details Formatted Diff Diff
updated with Balazs' comments (4.63 KB, patch) 2013-03-01 05:41 PST, Mariusz Grzegorczyk	andersca: review-	Details Formatted Diff Diff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.

Balazs Kelemen

Comment 1 2012-06-19 04:50:53 PDT

This is by design, I don't think we want to support plugins without plugin process.

Mariusz Grzegorczyk

Comment 2 2012-06-19 06:31:44 PDT

(In reply to comment #1) > This is by design, I don't think we want to support plugins without plugin process. So what for is this macro? I saw a lot of common code which checks if plugin process is on, and depends on it calls are made directly(same process) or via IPC. If you want to build webkit2 always with plugin process, maybe this setting should be removed, or event compilation shouldn't start.

Balazs Kelemen

Comment 3 2012-06-19 06:45:45 PDT

(In reply to comment #2) > (In reply to comment #1) > > This is by design, I don't think we want to support plugins without plugin process. > > So what for is this macro? I saw a lot of common code which checks if plugin process is on, and depends on it calls are made directly(same process) or via IPC. If you want to build webkit2 always with plugin process, maybe this setting should be removed, or event compilation shouldn't start. I guess the Mac port still supports disabling it and keeping it guarded could help in the future if we decide to change on this. Furthermore, the Qt build system does neither try to sanitize the custom build options you use, nor detect if you turn on something that is not supported, and we don't want to do this. I don't know what is the policy for Gtk.

Mariusz Grzegorczyk

Comment 4 2012-06-20 05:51:42 PDT

Anyway I think this kind of option can be useful. There is already infrastructure to do this in webcore(direct calls or via IPC depending on PLUGIN PROCESS macro). In my opinion each port should follow this rule, and support building with or without plugin process, or at least make port compilable with all parameters passed to build-webkit script.

Mariusz Grzegorczyk

Comment 5 2012-06-22 02:38:52 PDT

I discovered this issue while trying to support plugins to EFL's WK2. In the past I prepared such patch which works with/without plugin process. In the past there was no problem but now NetscapePluginModuleX11.cpp uses structs and functions which are defined only when PLUGIN_PROCESS is on. So because of this issue support for plugins in EFL's WK2 MUST use plugin process. What else Qt for linux also uses this file, so they cannot simply change their policy.

Balazs Kelemen

Comment 6 2012-06-23 05:50:36 PDT

(In reply to comment #5) > I discovered this issue while trying to support plugins to EFL's WK2. In the past I prepared such patch which works with/without plugin process. In the past there was no problem but now NetscapePluginModuleX11.cpp uses structs and functions which are defined only when PLUGIN_PROCESS is on. > So because of this issue support for plugins in EFL's WK2 MUST use plugin process. > What else Qt for linux also uses this file, so they cannot simply change their policy. Why do you think you are not allowed to change on this? If there are stuff necessary to build plugin code but not belongs to plugin process, you can fix the guards and make them independent from plugin process. Why don't you upload a patch that fix this? However, I wonder why would you disable plugin process. One of the biggest advantages of WebKit2 is stability, but you virtually lose this as soon as you load a plugin into the UI process. I don't expect that it's hard to port it to EFL, given that it's already a cross-platform feature.

Jonas Heinrich

Comment 7 2012-11-26 14:15:05 PST

Created attachment 176062 [details] NetscapePluginModuleX11 fails to compile I'm trying to build the latest webkitgtk nightly build 1.11.2 using the offical ArchLinux PKGBUILD: [...] cd "$srcdir/webkitgtk-$pkgver" PYTHON=/usr/bin/python2 ./configure --prefix=/usr \ --enable-introspection \ --disable-silent-rules \ --libexecdir=/usr/lib/webkitgtk2 \ --with-gstreamer=0.10 \ --with-gtk=2.0 \ --disable-webkit2 make all stamp-po [...] Now I'm getting the same error as described above (see attached log file). Is there any workaround to this? I can't get it compile and I don't know if I even need this Netscape plugin thing. Please help!

Carlos Garcia Campos

Comment 8 2012-11-27 00:27:12 PST

(In reply to comment #7) > Created an attachment (id=176062) [details] > NetscapePluginModuleX11 fails to compile > > I'm trying to build the latest webkitgtk nightly build 1.11.2 using the offical ArchLinux PKGBUILD: > > [...] > cd "$srcdir/webkitgtk-$pkgver" > PYTHON=/usr/bin/python2 ./configure --prefix=/usr \ > --enable-introspection \ > --disable-silent-rules \ > --libexecdir=/usr/lib/webkitgtk2 \ > --with-gstreamer=0.10 \ > --with-gtk=2.0 \ > --disable-webkit2 > make all stamp-po > [...] > > Now I'm getting the same error as described above (see attached log file). Is there any workaround to this? I can't get it compile and I don't know if I even need this Netscape plugin thing. > > Please help! I think this is a different bug because you are disabling webkit2 completely, this bug is about building with webkit2 but with plugin process disabled. I wonder why --libexecdir=/usr/lib/webkitgtk2 is passed and --disable-webkit2, though

Mariusz Grzegorczyk

Comment 9 2013-03-01 03:20:56 PST

Created attachment 190923 [details] patch

Balazs Kelemen

Comment 10 2013-03-01 04:59:46 PST

Comment on attachment 190923 [details] patch View in context: https://bugs.webkit.org/attachment.cgi?id=190923&action=review > Source/WebKit2/Shared/Plugins/Netscape/x11/NetscapePluginModuleX11.cpp:195 > } > > bool NetscapePluginModule::scanPlugin(const String& pluginPath) > +#else > +bool NetscapePluginModule::scanPlugin(const String& pluginPath, RawPluginMetaData& metaData) > +#endif > { > +#if ENABLE(PLUGIN_PROCESS) > RawPluginMetaData metaData; > - > +#endif > { > // Don't allow the plugin to pollute the standard output. > StdoutDevNullRedirector stdOutRedirector; Instead of using such ugly ifdefs, I would just factor out the code that you really need from here into a separate function and just call it when plugin process is disabled. Let's call it something like | queryPluginInfo(const String& pluginPath, RawPluginMetaData&) | which is just the block of stdOutRedirector from scanPlugin. It would look much better I think.

Mariusz Grzegorczyk

Comment 11 2013-03-01 05:41:30 PST

Created attachment 190944 [details] updated with Balazs' comments

Anders Carlsson

Comment 12 2013-09-25 08:39:44 PDT

Comment on attachment 190944 [details] updated with Balazs' comments ENABLE(PLUGIN_PROCESS) has been removed now, I believe this bug is invalid.

Note You need to log in before you can comment on or make changes to this bug.