[icedtea-web] RFC: allow alternate means of finding browsers

[Omair Majid]

Hi,

The attached patch allows netx to use a browser based on the BROWSER env 
variable or using xdg-open, as opposed to relying on user-supplied 
browser command.

A new configuration option deployment.browser.source is used to 
determine where to find the browser. If it is set to "PATH" (the 
default), the configuration option deployment.browser.path is used. If 
deployment.browser.source is set to "ENV" then the $BROWSER environment 
variable is used. If deployment.browser.source is set to "XDG" then the 
program xdg-open is used to launch the browser.

To remain compatible with other implementations, this option is only 
used when deployment.browser.path is null.

ChangeLog:
2010-12-22 Omair Majid  <omajid at redhat.com>

     * netx/net/sourceforge/jnlp/config/Defaults.java
     (getDefaults): Add new option.
     * netx/net/sourceforge/jnlp/config/DeploymentConfiguration.java:
     Add new constants BROWSER_FROM_PATH, BROWSER_FROM_ENV,
     BROWSER_FROM_FREEDESKTOP. Add new key KEY_BROWSER_SOURCE.
     * netx/net/sourceforge/jnlp/controlpanel/BrowserPanel.java: New
     file. Implements a panel to display the browser selection.
     * netx/net/sourceforge/jnlp/controlpanel/ComboItem.java: Make text
     private.
     * netx/net/sourceforge/jnlp/controlpanel/ControlPanel.java
     (createMainSettingsPanel): Add new browser tab.
     (createBrowserPanel): New method. Creates a JPanel to configure the
     browser.
     * netx/net/sourceforge/jnlp/resources/Messages.properties: Add
     CPTabBrowser, BPBrowser, BPBrowserEnv, BPBrowserFreeDesktop,
     and BPBrowserPath.
     * netx/net/sourceforge/jnlp/services/XBasicService.java
     (initialize): If command is null use configuration to find out
     whether the hardcoded path, or $BROWSER or xdg-open should be used.

Any thoughts or comments?

Cheers,
Omair
-------------- next part --------------
A non-text attachment was scrubbed...
Name: configure-browser-01.patch
Type: text/x-patch
Size: 13093 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20101222/4ee8e319/configure-browser-01.patch