- Jan 09, 2018
-
-
Jungshik Shin authored
38e3cf74 has a typo. Bug: None Change-Id: Id16f30ab5f2938bef567ef898f481696122a1dd3 TBR=hablich@chromium.org Reviewed-on: https://chromium-review.googlesource.com/856237 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
Jungshik Shin authored
The upstream also has this change to make AIX compiler happy. Bug: None Change-Id: I059ccc77a6b9aa0da9b4bab95103d2c10202f5a2 TBR=hablich@chromium.org Reviewed-on: https://chromium-review.googlesource.com/855583 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Dec 14, 2017
-
-
Jungshik Shin authored
In CLDR 32.0 (used in ICU 60), German AM/PM markers were changed to 'AM' and 'PM', but the change is controversial. For now, revert to the previous values per the upstream (CLDR 32.0.1 / ICU 60.2). See https://unicode.org/cldr/trac/ticket/10735 . Bug: 794737 Test: base_unittests --gtest_filter=TimeFormat*.*TimeOfDayDE Change-Id: Ifb715af8cf6d41803eaf612cbb8940dbc7851806 Reviewed-on: https://chromium-review.googlesource.com/826250 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Dec 13, 2017
-
-
Jungshik Shin authored
Bug: 794390 Change-Id: I31a1e622b38060c322be8086cc8aefdafb431a1b Reviewed-on: https://chromium-review.googlesource.com/822188 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Dec 12, 2017
-
-
Jungshik Shin authored
TBR=elwarence@chromium.org Bug: 792537 Change-Id: Idfc4fa258d0f9ffb6fd236183275ec4580a00f73 Reviewed-on: https://chromium-review.googlesource.com/809729 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Dec 08, 2017
-
-
Jungshik Shin authored
After data trimming, data/curr/en_GB.txt has empty "Currencies" block. This prevents a locale fallback chain from being triggered for currency symbols (en-GB => en-001 => en), leading all the currency symbols in en-GB to be the last resort value (ISO 3-letter currency code). To avoid that, trim_data.sh is revised to drop empty "Currencies" block entirely ('Missing' triggers a fallback chain, but 'empty' goes straight to the last resort.) Additional currency data fixes: 1. Serbian: RON and ROL display names are swapped. RON is the current currency and should be included with a shorter display name. 2. Add a few more en-* locale variants to the locale list for currency. Moreover, make_data_all.sh is updated to work wherever the Chromium source tree is. Finally, update the list of resources to remove from cast's ICU data and rebuild it with ICU 60. Added entries to cast-removed-resources.txt are data for locale-variants that are not useful in Cast. Bug: 791318 Test: See the bug Change-Id: Ib09ee36e3447557636d60d8aeabc7ca047718236 Reviewed-on: https://chromium-review.googlesource.com/810099 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Dec 07, 2017
-
-
Hans Wennborg authored
Recent Clang versions started taking into account that enums are signed on Windows when emitting these warnings. Bug: 792519 Change-Id: I32782dc34fc0480942038e4950266448e38d2eec Reviewed-on: https://chromium-review.googlesource.com/813495 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Nov 20, 2017
-
-
Mina Almasry authored
Our Cast Audio devices use a stripped down version of the android icudtl.dat. Essentially the patch_locale.sh has been modified to keep in things provided by Java, as cast doesn't have Java and needs there resources. On the other hand, a bunch of resources have been stripped from the icudtl.dat to reduce its size because we are very constrained on that. We have been using this icudtl.dat process for a long time now, and it's throughly validated. However, we used to generate this in our internal repo, which meant that we have to manually update it whenever chromium updates their icudtl.dat. This CL aims to intergrate the generation of the icudtl.dat in chromium, so we no longer have to manually update in our internal repo. Bug: internal b/28053340 Test: On cast audio device Change-Id: I881eb2ddf046fdd16fd7efbfb0d56877abfaf947 Reviewed-on: https://chromium-review.googlesource.com/773591 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Nov 09, 2017
-
-
Jungshik Shin authored
Cherry pick upstream fix for http://bugs.icu-project.org/trac/ticket/13462 which is http://bugs.icu-project.org/trac/changeset/40667 TBR=mathias@chromium.org Bug: v8:4743 Test: v8: test262/built-ins/RegExp/property-escapes/generated/* Change-Id: I3f94b6bf519bd395bec1f73e5f8d602f93910032 Reviewed-on: https://chromium-review.googlesource.com/760120 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Nov 07, 2017
-
-
Jungshik Shin authored
What's new in ICU 60.1: - Unicode 10.0 (including Emoji 5.0) - CLDR 32 (locale data; http://cldr.unicode.org/index/downloads/cldr-32 ) - Smaller rule file size for BreakIterator (no more reverse rules) - A new number formatting API - Malformed UTF-8 sequences are handled per W3C Encoding Standard - See http://site.icu-project.org/download/60 for more details. In addition, the following changes are made: * Add more locale variants for en,fr,de,it,pt,nl,ru,ar * Add the minimum locale data for Osage and Chakma * Apply locale data patches (Google and Chromium) * Apply build-related and break iterator patches: wpo.patch vscomp.patch data.build.win.patch data_symb.patch cjdict.patch khmer-dictbe.patch wordbrk.patch patches/data.build.patch * Update BUILD.gn and gyp files. * Data file size (25 ~ 130kB smaller than the latest 59.1) android/icudtl.dat 6499904 common/icudtl.dat 10170896 ios/icudtl.dat 6486240 The update steps are recorded in https://chromium.googlesource.com/chromium/deps/icu/+log/f88520f..1034c7e5 . Bug: 766816 Test: None (DEPS roll CL will have test specified). Change-Id: I9144bb37464eec1f282f40f9f2d28b6306511083 Reviewed-on: https://chromium-review.googlesource.com/756237 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Nov 03, 2017
-
-
Jungshik Shin authored
Bug: chromium:778300 Test: See bug commment 4 Change-Id: I1ae525323cb3354d3936a92e66cff266ebca2fb4 Reviewed-on: https://chromium-review.googlesource.com/738833 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Oct 11, 2017
-
-
Jungshik Shin authored
The location of zoneinfo directory has changed on macOS 10.13, Ubuntu 16, RHEL 7 and SuSe Linux 12. It results in the misdetection of the OS timezone by ICU. Cherry-picking the CLs for the following upstream bug fixes it. https://ssl.icu-project.org/trac/ticket/12770 Bug:754053,766916 Test: In Javascript console, the following should give the correct timezone. (new Intl.DateTimeFormat()).resolvedOptions().timeZone Change-Id: I711f4b27e73dc6855951055a601e80f711d34423 Reviewed-on: https://chromium-review.googlesource.com/710496 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Oct 10, 2017
-
-
Jungshik Shin authored
Bug:770452 Test: See the bug Change-Id: I5e19012a57507393670d68a7228bdb0831918997 Reviewed-on: https://chromium-review.googlesource.com/705920 Reviewed-by:
Mark Mentovai <mark@chromium.org>
-
- Aug 14, 2017
-
-
Scott Graham authored
Because the Fuchsia system bootdata.bin puts libicuuc.so and libicui18n.so in /system/lib already (and they don't match our build), and we have same-named .sos in component build, rename our copy. Bug: 732897, 743296 Change-Id: Ia195230c14d515ed14f9d90eb35aa87301eab601 Reviewed-on: https://chromium-review.googlesource.com/613506 Reviewed-by:
Nico Weber <thakis@chromium.org>
-
Jungshik Shin authored
1. Add DST abbreviation for Lord Howe timezone 2. Drop a few aliases for GBK/GB18030 These aliases are not listed in https://encoding.spec.whatwg.org/ 3. Fix Russian names for Hans/Hant (non-standalone). They're too long. 4. Add the abbreviated zone names for Asia/Pyongyang ICU data files get a bit smaller as below: Desktop: 10196720 -> 10196592 bytes Android: 6635312 -> 6635168 bytes iOS: 6623568 -> 6623360 bytes BUG=chromium:544228,chromium:339862,v8:6031 TEST=LayoutTests: fast/encoding/supported-encodings.html TBR=littledan@chromium.org Change-Id: I93881545a09485ff383a3630c85782c8754a9bd0 Reviewed-on: https://chromium-review.googlesource.com/599587 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Aug 03, 2017
-
-
Max Moroz authored
Bug: 751810 Change-Id: I377431074c25fbc8669cee311be304635953cc2a Reviewed-on: https://chromium-review.googlesource.com/600488 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Jul 27, 2017
-
-
Aaron Gable authored
R=jshin@chromium.org Change-Id: Ia650d3b9699e0cf074265c61a8656f65bc4c7def Reviewed-on: https://chromium-review.googlesource.com/587912 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Jul 19, 2017
-
-
Jungshik Shin authored
Blink does not need it any more. Drop UTF-32 converters and aliases. Save 320 bytes in ICU data. icuuc.so will be slightly smaller, too. BUG=chromium:417850 TEST=None Change-Id: Ib4fecc3fcc905c974e68a652e7f4722bfd271129 Reviewed-on: https://chromium-review.googlesource.com/576872 Reviewed-by:
Joshua Bell <jsbell@chromium.org>
-
- Jul 18, 2017
-
-
Jungshik Shin authored
The following locale variants are added to the ICU. Only source/data/locales are added and other locale categories are left alone. - es-* variants - en-SG and en-HK - Arabic variants whose NumberElements are different from ar. Data file size changes: - desktop: 10192200 - 10178576 = 13624 (adding es variants) 10197040 - 10192200 = 4840 (adding ar variants) - Android: 6635632 - 6616864 = 18768 - iOS: 6623888 - 6605152 = 18736 Bug:chromium:732762 Test:See the bug report and comment 13 TBR=littledan@chromium.org Change-Id: I3e87be3413e22aa49f24f81b0e25bd6e4187c0d4 Reviewed-on: https://chromium-review.googlesource.com/576113 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Jul 17, 2017
-
-
Jungshik Shin authored
They're supported by Google Translate and their names need to be shown in the Translate UI. Hmong (hmn) Javanese (jv) Luxembourgish (lb) Samoan (sm) The size impact for desktop Chrome is about 3kB. ( 10,175,056 => 10,178,576) BUG=733398 TEST=See the bug TBR=yyushkina@chromium.org Change-Id: I9ca232343847a9a95c1dabacb6336bd43ba39849 Reviewed-on: https://chromium-review.googlesource.com/574771 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
Jungshik Shin authored
http://bugs.icu-project.org/trac/ticket/13277 Also fix README.chromium file to refer to 59.1 instead of 58.1. BUG=chromium:734093 TEST=None (upstream has a test but not included in the Chrome's copy of ICU) TBR=avi@chromium.org Change-Id: I0177d80d7c94199fdea107f891a922b67f0d8481 Reviewed-on: https://chromium-review.googlesource.com/574935 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- Jun 02, 2017
-
-
Tom Anderson authored
BUG=729032 R=jshin@chromium.org Review-Url: https://codereview.chromium.org/2921843003 .
-
- Jun 01, 2017
-
-
Jungshik Shin authored
Bug:chromium:726654 Test: gn gen works without an error Review-Url: https://codereview.chromium.org/2913423002 .
-
- May 31, 2017
-
-
Jungshik Shin authored
This is necessary for Chrome OS to calculate sunset time. Bug:chromium:726654 Test: With https://codereview.chromium.org/2887913004 patched in, 'gn Change-Id: Icf6a8cd833f8afcd1a2d0ce874cf1c8c92fcba44 gen <out_dir> --check' does not complain. TBR=afakhry@chromium.org Reviewed-on: https://chromium-review.googlesource.com/520323 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- May 18, 2017
-
-
Scott Graham authored
Without a Linux-ish define for Fuchsia, neither U_WCHAR_IS_UTF16 nor U_WCHAR_IS_UTF32 are defined, resulting in warnings in ICU code when compiled on Fuchsia. Filed upstream at http://www.icu-project.org/trac/ticket/13202. BUG=706592 R=jshin@chromium.org Review-Url: https://codereview.chromium.org/2885143008 .
-
- May 16, 2017
-
-
Jungshik Shin authored
http://www.icu-project.org/trac/ticket/12333 http://www.icu-project.org/trac/ticket/13189 http://www.icu-project.org/trac/ticket/12635 - patches/ucase_utf8.patch - patches/ucurr_locale.patch - patches/collator_range.patch Bug=chromium:722124 TEST=See the test TBR=inferno@chromium.org Change-Id: I3e754734a1c1121c8059a386ca1b5d0f004adb79 Reviewed-on: https://chromium-review.googlesource.com/506592 Reviewed-by:
Abhishek Arya <inferno@chromium.org> Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- May 15, 2017
-
-
Michael Achenbach authored
Also add support for host_byteorder logic which is introduced in - https://codereview.chromium.org/2815453004/ R=adamk@chromium.org, dpranke@chromium.org, machenbach@chromium.org BUG=706728 Review-Url: https://codereview.chromium.org/2812173002 .
-
- May 14, 2017
-
-
Jungshik Shin authored
* Highlights: - Emoji 5.0 data (partial; Emoji_Component property not included) - CLDR 31.0.1 (http://blog.unicode.org/2017/03/cldr-version-31-released.html) UTC and GMT are treated as distinct) - New case mapping API for styled text - C++ 11 is required - char16_t for UChar (UTF-16) - Source code is in UTF-8 * Size changes common: 10,130,560 => 10,175,056 android: 6,573,872 => 6,616,864 iOS: 6,562,352 => 6,605,152 On top of ICU 59.1 from the upstream, the following changes were applied. See https://chromium.googlesource.com/chromium/deps/icu/+log/chromium/59staging - Fix C++ 11 string literal assignment issue (upstream bug: 13192) - Fix C4229 warning by MSVC - Apply utf32.patch and include unistr.h in fuzzer_util - Update ICU data files - Fix wpo.patch - Apply Google locale patch and locale1.patch - update readme - Apply breakiterator related patches - Apply and update wpo.patch - Drop unused patch, apply data.build.win.patch, update README.chromium - Add /utf-8 flag for Windows/Visual Studio - Update BUILD.gn for UChar, stubdata and apply data_sym.patch - use stubdata.cpp instead of stubdata.c in icu.gyp - Update icu.gyp* files for v8 - Update BUILD.gn, apply data.build.patch and vscomp.patch - Add new files in ICU 59.1 - Get a fresh copy of ICU 59.1 from the upstream - Update update.sh script TBR=drott@chromium.org, yangguo@chromium.org Bug:699469 TEST: layout tests, all unittests, browser tests Change-Id: Ie1e77323aa0c7f872153680c4deca6471a771a5c Reviewed-on: https://chromium-review.googlesource.com/505173 Reviewed-by:
Jungshik Shin <jshin@chromium.org>
-
- May 05, 2017
-
-
Jungshik Shin authored
Add ios/icudtl.dat and ios/patch_locale.sh. Update README.chromium and BUILD.gn accordingly. Update scripts/copy_data.sh to take (ios|common|android). At the moment, iOS data is almost identical to that of Android, but in the future more cuts may be made (e.g. dictionary data for breakiterator). Bug: 718955 TEST: iOS Chrome works as before. Review-Url: https://codereview.chromium.org/2743123002 .
-
- Apr 25, 2017
-
-
Max Moroz authored
BUG=539572 R=jshin@chromium.org Review-Url: https://codereview.chromium.org/2835993002 .
-
- Apr 11, 2017
-
-
Jungshik Shin authored
See http://mm.icann.org/pipermail/tz-announce/2017-March/000046.html for details. - Haiti resumed observing DST. Data file size changes: android/icudtl.dat 6573792 => 6573872 common/icudt*.dat 10130480 => 10130560 BUG=473288 TEST=Try the following in d8 or Chrome's JS console: > (new Date("3/12/2016 12:00Z")).toLocaleString("en", {timeZone: "America/Port-au-Prince"}) "3/12/2016, 7:00:00 AM" > (new Date("3/12/2017 12:00Z")).toLocaleString("en", {timeZone: "America/Port-au-Prince"}) "3/12/2017, 8:00:00 AM" Review-Url: https://codereview.chromium.org/2807343004 .
-
Jungshik Shin authored
Delete empty units,units{Narrow,Short} blocks after trimming units data. Empty units* blocks in en_GB and a few other locales after trimming causes ICU to fail to fall back to get the duration data for those locales. In addition, fix source/data/translit/root_subset.txt. Rule*Ids block has to be present even though it's empty. When dropping Hans-Hant transform rules, root_subset.txt was changed to be completely empty, which broke "components_unittests --g_test_filter=AutofillProfileComparato*" . With these changes, regenerate ICU data files. The size is slightly smaller. android/icudtl.dat 6573872 => 6573792 common/icudt*dat 10130560 => 10130480 BUG=707515,677043,684609 TEST=components_unittests --gtest_filter=AutofillProfileComparato* TEST=ui_base_unittests --gtest_filter=L10nUtilTest.TimeDurationForm* R=derat@chromium.org Review-Url: https://codereview.chromium.org/2812943003 .
-
- Mar 16, 2017
-
-
Jungshik Shin authored
See http://mm.icann.org/pipermail/tz-announce/2017-February/000045.html for details. - Mongolia stops observing DST. - America/Punta_Arenas: a new timezone for Soutthern Chile android/icudtl.dat 6573024 => 6573872 common/icudtb.dat 10129712 => 10130560 common/icudtl.dat 10129712 => 10130560 BUG=473288 TEST=Try the following in d8 or Chrome's JS console: > (new Date("4/4/2017 12:00Z")).toLocaleString("en", {timeZone: "Asia/Ulaanbaatar"}) "4/4/2016, 8:00:00 PM" > (new Date("4/4/2016 12:00Z")).toLocaleString("en", {timeZone: "Asia/Ulaanbaatar"}) "4/4/2016, 9:00:00 PM" TBR=mark Review-Url: https://codereview.chromium.org/2755833002 .
-
Jungshik Shin authored
The file was mistakenly dropped from https://codereview.chromium.org/2747173004/ BUG=684609 TBR=riesa@chromium.org Review-Url: https://codereview.chromium.org/2750943005 .
-
- Mar 15, 2017
-
-
Jungshik Shin authored
Summary of data size decrease: android/icudtl.dat 6610128 -> 6573024 bytes common/icudtb.dat 10166816 -> 10129712 bytes common/icudtl.dat 10166816 -> 10129712 bytes Patch by riesa@chromium.org BUG=684609 R=jshin@chromium.org Review-Url: https://codereview.chromium.org/2747173004 .
-
- Mar 07, 2017
-
-
Jungshik Shin authored
BUG=None Review-Url: https://codereview.chromium.org/2736013003 .
-
Jungshik Shin authored
Fix the path for Khmer dictionary and word_ja in data_files_to_preserve.txt. Add icu4c to the upstream repository path. BUG=None Review-Url: https://codereview.chromium.org/2732393002 .
-
- Feb 21, 2017
-
-
Jungshik Shin authored
Size increase for affected data files is as follows: android/icudtl.dat 6573776 -> 6610128 bytes common/icudtb.dat 10130464 -> 10166816 bytes common/icudtl.dat 10130464 -> 10166816 bytes This CL supercedes CL 2328013002. CL by riesa@chromium.org. BUG=684609 R=jshin@chromium.org Review-Url: https://codereview.chromium.org/2652023002 .
-
- Jan 27, 2017
-
-
mmoroz authored
Ported from OSS-Fuzz: https://github.com/google/oss-fuzz/pull/304/files. TBR=aizatsky@chromium.org, jshin@chromium.org BUG= Review-Url: https://codereview.chromium.org/2650653003 .
-
- Dec 12, 2016
-
-
Tomasz Śniatowski authored
https://codereview.chromium.org/2504463002 added suppresion of deprecation warnings in an "if clang" block, but gcc also warns about these, as can be seen on Chrome-Android waterfall for example. Fix by moving the supression to an if linux||android block. BUG=21515 R=thakis@chromium.org Review URL: https://codereview.chromium.org/2571433002 . Patch from Tomasz Śniatowski <tsniatowski@opera.com>.
-