https://bugs.webkit.org/show_bug.cgi?id=99583
Patch by Jussi Kukkonen <jussi.kukkonen@intel.com> on 2012-10-18
Reviewed by Sam Weinig.
Add a platform specific launch options initializer. This enables
fixing bug 99297 "[EFL] Make plugin process debugging easier
(PLUGIN_PROCESS_COMMAND_PREFIX)".
* UIProcess/Plugins/PluginProcessProxy.cpp:
(WebKit::PluginProcessProxy::PluginProcessProxy):
* UIProcess/Plugins/PluginProcessProxy.h:
(PluginProcessProxy):
* UIProcess/Plugins/mac/PluginProcessProxyMac.mm:
(WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
(WebKit):
* UIProcess/Plugins/qt/PluginProcessProxyQt.cpp:
(WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
(WebKit):
* UIProcess/Plugins/unix/PluginProcessProxyUnix.cpp:
(WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
(WebKit):
git-svn-id: https://svn.webkit.org/repository/webkit/trunk@131717
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2012-10-18 Jussi Kukkonen <jussi.kukkonen@intel.com>
+
+ Add PluginProcessProxy::platformInitializeLaunchOptions
+ https://bugs.webkit.org/show_bug.cgi?id=99583
+
+ Reviewed by Sam Weinig.
+
+ Add a platform specific launch options initializer. This enables
+ fixing bug 99297 "[EFL] Make plugin process debugging easier
+ (PLUGIN_PROCESS_COMMAND_PREFIX)".
+
+ * UIProcess/Plugins/PluginProcessProxy.cpp:
+ (WebKit::PluginProcessProxy::PluginProcessProxy):
+ * UIProcess/Plugins/PluginProcessProxy.h:
+ (PluginProcessProxy):
+ * UIProcess/Plugins/mac/PluginProcessProxyMac.mm:
+ (WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
+ (WebKit):
+ * UIProcess/Plugins/qt/PluginProcessProxyQt.cpp:
+ (WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
+ (WebKit):
+ * UIProcess/Plugins/unix/PluginProcessProxyUnix.cpp:
+ (WebKit::PluginProcessProxy::platformInitializeLaunchOptions):
+ (WebKit):
+
2012-10-18 Byungwoo Lee <bw80.lee@samsung.com>
[WK2] Update test expectation for the python message test after r131700
{
ProcessLauncher::LaunchOptions launchOptions;
launchOptions.processType = ProcessLauncher::PluginProcess;
-#if PLATFORM(MAC)
- launchOptions.architecture = pluginInfo.pluginArchitecture;
- launchOptions.executableHeap = PluginProcessProxy::pluginNeedsExecutableHeap(pluginInfo);
-#if HAVE(XPC)
- launchOptions.useXPC = false;
-#endif
-#endif
+
+ platformInitializeLaunchOptions(launchOptions, pluginInfo);
m_processLauncher = ProcessLauncher::create(this, launchOptions);
}
void applicationDidBecomeActive();
#endif
+ static void platformInitializeLaunchOptions(ProcessLauncher::LaunchOptions&, const PluginModuleInfo& pluginInfo);
void platformInitializePluginProcess(PluginProcessCreationParameters& parameters);
// The plug-in host process manager.
return true;
}
+void PluginProcessProxy::platformInitializeLaunchOptions(ProcessLauncher::LaunchOptions& launchOptions, const PluginModuleInfo& pluginInfo)
+{
+ launchOptions.architecture = pluginInfo.pluginArchitecture;
+ launchOptions.executableHeap = PluginProcessProxy::pluginNeedsExecutableHeap(pluginInfo);
+#if HAVE(XPC)
+ launchOptions.useXPC = false;
+#endif
+}
+
void PluginProcessProxy::platformInitializePluginProcess(PluginProcessCreationParameters& parameters)
{
// For know only Flash is known to behave with asynchronous plug-in initialization.
class PluginProcessCreationParameters;
+void PluginProcessProxy::platformInitializeLaunchOptions(ProcessLauncher::LaunchOptions& launchOptions, const PluginModuleInfo& pluginInfo)
+{
+}
+
void PluginProcessProxy::platformInitializePluginProcess(PluginProcessCreationParameters&)
{
}
namespace WebKit {
+void PluginProcessProxy::platformInitializeLaunchOptions(ProcessLauncher::LaunchOptions& launchOptions, const PluginModuleInfo& pluginInfo)
+{
+}
+
void PluginProcessProxy::platformInitializePluginProcess(PluginProcessCreationParameters&)
{
}