[rfc][icedtea-1.4] Cache viewer cleanup backport

[Jacob Wisor]

Hello!

This patch is a backport of the cache viewer cleanup changeset 
(http://icedtea.classpath.org/hg/icedtea-web/?cmd=changeset;node=c797a2c1329b) 
for 1.4 which has already been pushed into trunk.

@Jiri
I was not sure about one thing here:

--- a/netx/net/sourceforge/jnlp/controlpanel/CachePane.java
+++ b/netx/net/sourceforge/jnlp/controlpanel/CachePane.java
[...]
@@ -260,13 +272,137 @@
[...]
+            private void updateRecentlyUsed(final File f) {
+                final File recentlyUsedFile = new File(location + 
File.separator + "recently_used");

Since you have touched net.sourceforge.jnlp.cache.CacheLRUWrapper due to the XDG 
Specification (which is unsupported on 1.4) changeset for 1.5 this line could or 
should have read

final File recentlyUsedFile = new File(location + File.separator + 
CacheLRUWrapper.CACHE_INDEX_FILE_NAME);

In 1.4, CacheLRUWrapper.CACHE_INDEX_FILE_NAME is a "recently_used" constant 
indeed. But, the CacheLRUWrapper enum has default instead of public access (as 
in 1.5), so CacheLRUWrapper.CACHE_INDEX_FILE_NAME is not accessible from 
CachePane either, although this member constant itself has public static access. 
In 1.5, CacheLRUWrapper has gotten public access since that XDG Specification 
changeset, so I was not sure whether it would be permissible to change 
CacheLRUWrapper's access to public in 1.4 too or rather to use the 
"recently_used" literal as this has been the case in 1.4 ever since. In fear of 
that some of the CacheLRUWrapper enum's vital data could possibly get exposed, I 
have resorted to using the literal.
So, what do you think?

Thank you for any reviewing efforts. :-)

Jacob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: CacheViewer.patch
Type: text/x-patch
Size: 39508 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/distro-pkg-dev/attachments/20140206/f556dc52/CacheViewer-0001.patch