8228465: HOST locale provider holds wrong era name for GregorianCalendar in US locale

Reviewed-by: lancea
This commit is contained in:
Naoto Sato 2019-07-26 13:32:59 -07:00
parent d711c4e18e
commit 11ab995e6c
3 changed files with 32 additions and 7 deletions

View File

@ -2205,7 +2205,8 @@ public abstract class Calendar implements Serializable, Cloneable, Comparable<Ca
}
String calendarType = getCalendarType();
if (style == ALL_STYLES || isStandaloneStyle(style) || isNarrowFormatStyle(style)) {
if (style == ALL_STYLES || isStandaloneStyle(style) || isNarrowFormatStyle(style) ||
field == ERA && (style & SHORT) == SHORT) {
Map<String, Integer> map;
map = CalendarDataUtility.retrieveFieldValueNames(calendarType, field, style, locale);

View File

@ -28,6 +28,8 @@ import sun.util.locale.provider.LocaleProviderAdapter;
public class LocaleProviders {
private static final boolean IS_WINDOWS = System.getProperty("os.name").startsWith("Windows");
public static void main(String[] args) {
String methodName = args[0];
@ -76,6 +78,10 @@ public class LocaleProviders {
bug8220227Test();
break;
case "bug8228465Test":
bug8228465Test();
break;
default:
throw new RuntimeException("Test method '"+methodName+"' not found.");
}
@ -106,7 +112,7 @@ public class LocaleProviders {
static void bug7198834Test() {
LocaleProviderAdapter lda = LocaleProviderAdapter.getAdapter(DateFormatProvider.class, Locale.US);
LocaleProviderAdapter.Type type = lda.getAdapterType();
if (type == LocaleProviderAdapter.Type.HOST && System.getProperty("os.name").startsWith("Windows")) {
if (type == LocaleProviderAdapter.Type.HOST && IS_WINDOWS) {
DateFormat df = DateFormat.getDateInstance(DateFormat.FULL, Locale.US);
String date = df.format(new Date());
if (date.charAt(date.length()-1) == ' ') {
@ -133,7 +139,7 @@ public class LocaleProviders {
// This test assumes Windows localized language/country display names.
static void bug8010666Test() {
if (System.getProperty("os.name").startsWith("Windows")) {
if (IS_WINDOWS) {
NumberFormat nf = NumberFormat.getInstance(Locale.US);
try {
double ver = nf.parse(System.getProperty("os.version"))
@ -215,7 +221,7 @@ public class LocaleProviders {
}
static void bug8013903Test() {
if (System.getProperty("os.name").startsWith("Windows")) {
if (IS_WINDOWS) {
Date sampleDate = new Date(0x10000000000L);
String hostResult = "\u5e73\u6210 16.11.03 (Wed) AM 11:53:47";
String jreResult = "\u5e73\u6210 16.11.03 (\u6c34) \u5348\u524d 11:53:47";
@ -241,7 +247,7 @@ public class LocaleProviders {
}
static void bug8027289Test(String expectedCodePoint) {
if (System.getProperty("os.name").startsWith("Windows")) {
if (IS_WINDOWS) {
char[] expectedSymbol = Character.toChars(Integer.valueOf(expectedCodePoint, 16));
NumberFormat nf = NumberFormat.getCurrencyInstance(Locale.CHINA);
char formatted = nf.format(7000).charAt(0);
@ -255,7 +261,7 @@ public class LocaleProviders {
}
static void bug8220227Test() {
if (System.getProperty("os.name").startsWith("Windows")) {
if (IS_WINDOWS) {
Locale l = new Locale("xx","XX");
String country = l.getDisplayCountry();
if (country.endsWith("(XX)")) {
@ -264,4 +270,20 @@ public class LocaleProviders {
}
}
}
static void bug8228465Test() {
LocaleProviderAdapter lda = LocaleProviderAdapter.getAdapter(CalendarNameProvider.class, Locale.US);
LocaleProviderAdapter.Type type = lda.getAdapterType();
if (type == LocaleProviderAdapter.Type.HOST && IS_WINDOWS) {
var names = new GregorianCalendar()
.getDisplayNames(Calendar.ERA, Calendar.SHORT_FORMAT, Locale.US);
if (!names.keySet().contains("AD") ||
names.get("AD").intValue() != 1) {
throw new RuntimeException(
"Short Era name for 'AD' is missing or incorrect");
} else {
System.out.println("bug8228465Test succeeded.");
}
}
}
}

View File

@ -25,7 +25,7 @@
* @test
* @bug 6336885 7196799 7197573 7198834 8000245 8000615 8001440 8008577
* 8010666 8013086 8013233 8013903 8015960 8028771 8054482 8062006
* 8150432 8215913 8220227
* 8150432 8215913 8220227 8228465
* @summary tests for "java.locale.providers" system property
* @library /test/lib
* @build LocaleProviders
@ -154,6 +154,8 @@ public class LocaleProvidersRun {
testRun("HOST", "bug8220227Test", "", "", "");
}
//testing 8228465 fix. (Windows only)
testRun("HOST", "bug8228465Test", "", "", "");
}
private static void testRun(String prefList, String methodName,