Regression in itw from Tue Mar 26

[Jiri Vanek]

Hi, commit

  changeset:   656:1c26ce1e1cb3
  tag:         tip
  user:        Adam Domurad <adomurad at redhat.com>
  date:        Tue Mar 26 14:57:33 2013 -0400
  summary:     Integration of unsigned applet confirmation dialogue.

broke the	

CodeBaseClassLoaderTest.testNullFileSecurityDescApplication test

The affecting hunk was

@@ -596,9 +604,9 @@

          JARDesc jars[] = resources.getJARs();

-        if (jars == null || jars.length == 0) {
+        if (jars.length == 0) {

-            boolean allSigned = true;
+            boolean allSigned = (loaders.length > 1) /* has extensions */;
              for (int i = 1; i < loaders.length; i++) {
                  if (!loaders[i].getSigning()) {
                      allSigned = false;

of diff -r bb971f25eb42 -r 1c26ce1e1cb3 netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java

and the responsible line was

-            boolean allSigned = true;
+            boolean allSigned = (loaders.length > 1) /* has extensions */;

An I must say I'm hesitating with fix. Just use always true? or  use (loaders.length > 0) which is 
actually the same.

Why the change at all?

J.

see http://10.34.2.200/icedtea-web-dailyreport/ICWDR_1364327226/index.html for full info