Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(181)

Unified Diff: test/test262/test262.status

Issue 1812673005: Use ICU case conversion/transliterator for case conversion behind a flag (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Yang's comment addressed - return right away for no-change Created 4 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« src/runtime/runtime-i18n.cc ('K') | « test/intl/testcfg.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/test262/test262.status
diff --git a/test/test262/test262.status b/test/test262/test262.status
index 85c6d276b12a2d0128992e95a1610b505affbf72..e130c62c0f3cc328e11fc3c5ec5188807df5a15d 100644
--- a/test/test262/test262.status
+++ b/test/test262/test262.status
@@ -139,14 +139,16 @@
'intl402/NumberFormat/11.1.1_1': [FAIL],
# https://code.google.com/p/v8/issues/detail?id=4476
- 'built-ins/String/prototype/toLocaleLowerCase/special_casing_conditional': [FAIL],
- 'built-ins/String/prototype/toLocaleLowerCase/supplementary_plane': [FAIL],
+ # The bug is fixed but behind a flag, --icu_case_mapping.
'built-ins/String/prototype/toLowerCase/special_casing_conditional': [FAIL],
'built-ins/String/prototype/toLowerCase/supplementary_plane': [FAIL],
- 'built-ins/String/prototype/toLocaleUpperCase/supplementary_plane': [FAIL],
'built-ins/String/prototype/toUpperCase/supplementary_plane': [FAIL],
# https://code.google.com/p/v8/issues/detail?id=4477
+ # The bug is fixed but behind a flag, --icu_case_mapping.
+ 'built-ins/String/prototype/toLocaleUpperCase/supplementary_plane': [FAIL],
+ 'built-ins/String/prototype/toLocaleLowerCase/supplementary_plane': [FAIL],
+ 'built-ins/String/prototype/toLocaleLowerCase/special_casing_conditional': [FAIL],
'intl402/String/prototype/toLocaleLowerCase/special_casing_Azeri': [FAIL],
'intl402/String/prototype/toLocaleLowerCase/special_casing_Lithuanian': [FAIL],
'intl402/String/prototype/toLocaleLowerCase/special_casing_Turkish': [FAIL],
@@ -423,6 +425,22 @@
'built-ins/String/prototype/normalize/return-normalized-string': [SKIP],
'built-ins/String/prototype/normalize/return-normalized-string-from-coerced-form': [SKIP],
'built-ins/String/prototype/normalize/return-normalized-string-using-default-parameter': [SKIP],
+
+ # Case-conversion is not fully compliant to the Unicode spec with i18n off.
+ 'built-ins/String/prototype/toLocaleLowerCase/special_casing_conditional': [FAIL],
+ 'built-ins/String/prototype/toLocaleLowerCase/supplementary_plane': [FAIL],
+ 'built-ins/String/prototype/toLowerCase/special_casing_conditional': [FAIL],
+ 'built-ins/String/prototype/toLowerCase/supplementary_plane': [FAIL],
+ 'built-ins/String/prototype/toLocaleUpperCase/supplementary_plane': [FAIL],
+ 'built-ins/String/prototype/toUpperCase/supplementary_plane': [FAIL],
+
+ # Locale-sensitive case-conversion is not available with i18n off.
+ 'intl402/String/prototype/toLocaleLowerCase/special_casing_Azeri': [FAIL],
+ 'intl402/String/prototype/toLocaleLowerCase/special_casing_Lithuanian': [FAIL],
+ 'intl402/String/prototype/toLocaleLowerCase/special_casing_Turkish': [FAIL],
+ 'intl402/String/prototype/toLocaleUpperCase/special_casing_Azeri': [FAIL],
+ 'intl402/String/prototype/toLocaleUpperCase/special_casing_Lithuanian': [FAIL],
+ 'intl402/String/prototype/toLocaleUpperCase/special_casing_Turkish': [FAIL],
}], # no_i18n == True
['arch == arm or arch == mipsel or arch == mips or arch == arm64 or arch == mips64 or arch == mips64el', {
« src/runtime/runtime-i18n.cc ('K') | « test/intl/testcfg.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698