RFC: Netx - Use version based download protocol for JNLP files

[Omair Majid]

Hi,

This patch makes Netx use the version based download protocol for JNLP 
files.

Netx tries to use the version based download protocol whenever possible 
to download files. This wasn't being done for JNLP file that were 
specified as extensions by other JNLP files. This patch fixes the issue 
making JNLP applications like aerith[1] work. This is also required to 
run JavaFX applications.

While adding that support, I noticed that function arguments that 
involved Urls and versions were inconsistent: some used function(URL, 
Version, blah) while others used function(URL, blah, Version). I have 
tried to fix them to be function(URL, Version, blah) consistently.

ChangeLog:
2009-07-10  Omair Majid  <omajid at redhat.com>

   * rt/net/sourceforge/jnlp/JNLPFile.java
   (JNLPFile): Delegate to the Version-based constructor.
   (JNLPFile): New constructor.
   (JNLPFile): Modified to take an additional version argument used in
   downloading the JNLP file.
   (openURL): Take an additional version argument and use when
   downloading the URL.
   * rt/net/sourceforge/jnlp/Launcher.java
   (toFile): Use the new JNLPFile constructor.
   * rt/net/sourceforge/jnlp/cache/Resource.java
   (Resource): Rearrange argument order.
   (getResource): Likewise. Fix parameters to constructor.
   * rt/net/sourceforge/jnlp/cache/ResourceTracker.java
   (addResource): Fix arguments to Resource.getResource.
   * rt/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
   (getInstance): Take additional version argument and useit when
   creating a JNLPFile.
   (initializeExtensions): Use the extension version when requesting a
   JNLPClassLoader.


Any comments?

Cheers,
Omair

[1] https://aerith.dev.java.net/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: netx-versioned-jnlp.patch
Type: text/x-patch
Size: 7497 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20090710/8396ada0/netx-versioned-jnlp.patch