[RFC][icedtea-web]: Set plugin JVM options in itw-settings

Saad Mohammad smohammad at redhat.com
Mon Oct 29 12:30:41 PDT 2012


Hi,

The patch attached contains a new feature allowing users to set plugin JVM
options through itw-settings. The patch also cleans up itw-settings UI by
organizing debugging, desktop integration, and security tab into a single tab
named Advanced Settings.

A simple way of testing this feature is by adding '-verbose' within settings and
launching any applet in the web-browser.

You'll also note the patch removes extra spaces after it retrieves jvm options
from itw-settings. This is because extra spaces can sometimes break the launch
of the plugin.

Thanks.

CHANGELOG
========================================================================
2012-10-29  Saad Mohammad  <smohammad at redhat.com>

	Added new options in itw-settings which allows users to set JVM
	options when plugin is initialized.
	* netx/net/sourceforge/jnlp/config/Defaults.java (getDefaults):
	Added default for DeploymentConfiguration.KEY_PLUGIN_JVM_OPTIONS.
	* netx/net/sourceforge/jnlp/config/DeploymentConfiguration.java:
	Added new property (KEY_PLUGIN_JVM_OPTIONS) that stores the value of
	JVM plugin options.
	* netx/net/sourceforge/jnlp/controlpanel/AdvancedSettingsPanel.java:
	New advanced settings panel containing a single organized view of
	older tabs and plugin settings.
	* netx/net/sourceforge/jnlp/controlpanel/ControlPanel.java
(createMainSettingsPanel):
	Added advanced settings to the list of tabs and moved debugging,
	desktop integration, and security from the tab list to the new
	panel.
	* netx/net/sourceforge/jnlp/controlpanel/PluginPanel:
	New plugin settings panel.
	* netx/net/sourceforge/jnlp/resources/Messages.properties:
	Added a new items (CPPluginJVMOptions, CPHeadPlugin,
	CPTabAdvancedSettings) and removed unused ones.
	* plugin/icedteanp/IcedTeaNPPlugin.cc:
	(plugin_start_appletviewer): Adds JVM options to the command that
	initalizes it.
	(get_JVM_options): Returns JVM options set in itw-settings.
	(trim_char): Cleans chars by removing extra spaces and new lines.
	* plugin/icedteanp/IcedTeaNPPlugin.h:
	Declaration of two new methods.

-- 
Cheers,
Saad Mohammad

-------------- next part --------------
A non-text attachment was scrubbed...
Name: changelog0-1.patch
Type: text/x-patch
Size: 1641 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20121029/56f21c29/changelog0-1.patch 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: implementation0-1.patch
Type: text/x-patch
Size: 14875 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20121029/56f21c29/implementation0-1.patch 


More information about the distro-pkg-dev mailing list