8006667: Merge issue: Profile attribute need to be examined before custom attributes

Swap profile checking and FXHelper checking

Reviewed-by: alanb
This commit is contained in:
David Holmes 2013-01-22 20:04:15 -05:00
parent 8b799721ae
commit 26936d0600

View File

@ -413,15 +413,6 @@ public enum LauncherHelper {
if (mainValue == null) {
abort(null, "java.launcher.jar.error3", jarname);
}
/*
* Hand off to FXHelper if it detects a JavaFX application
* This must be done after ensuring a Main-Class entry
* exists to enforce compliance with the jar specification
*/
if (mainAttrs.containsKey(
new Attributes.Name(FXHelper.JAVAFX_APPLICATION_MARKER))) {
return FXHelper.class.getName();
}
/*
* If this is not a full JRE then the Profile attribute must be
@ -444,6 +435,16 @@ public enum LauncherHelper {
}
}
/*
* Hand off to FXHelper if it detects a JavaFX application
* This must be done after ensuring a Main-Class entry
* exists to enforce compliance with the jar specification
*/
if (mainAttrs.containsKey(
new Attributes.Name(FXHelper.JAVAFX_APPLICATION_MARKER))) {
return FXHelper.class.getName();
}
return mainValue.trim();
} catch (IOException ioe) {
abort(ioe, "java.launcher.jar.error1", jarname);