[RFC] netx: only create one instance of JNLPClassLoader for entire application

[Omair Majid]

On 07/08/2010 01:56 PM, Deepak Bhole wrote:
> * Omair Majid<omajid at redhat.com>  [2010-07-08 09:59]:
>> Hi,
>>
>> The attached patch modifies the JNLPClassLoader so only one instance
>> of the loader is created per application. This patch also makes the
>> classloader keep track of each JNLP file and the resources it loads.
>> For applets, it only keeps track of the main applet JNLP (the
>> PluginBridge) and uses that.
>>
>> Any comments?
>>
>> Cheers,
>> Omair
>
> We already use a single classloader (by merging extension loader paths
> into the base loader)...
>
> What bug required this change?
>

We merge the paths, but lose all the other information. Consider this: 
anyplace where the current code does a file.foo(), we dont know which 
JNLPFile file actually is.

Also, this updated version (which knows about the main application JNLP 
and the extension JNLPs) will make it easier to implement the new cache 
for netx that I have been planning. During the application's startup, 
when the Jars are being downloaded and verified, we can ensure that all 
of the application's jars are treated the same way and we present a 
consistent cache to the application. In the current implementation 
sincle multiple classloaders are created, each JNLPClassLoader considres 
the jars individually for updates and caching.

Still, because of how this patch affects the ClassLoader, I would 
appreciate it people could test it out for a bit before it commit it.

Thanks,
Omair