- Dec 15, 2014
-
-
Jungshik Shin (jungshik at google) authored
To have keyboard layouts for ku and ckb, we need to have the minimal locale data for them - native display name, writing direction, and exemplar characters. These are added and the icu data was rebuilt. The Android icu data was not rebuilt because Android Chrome does not use them. BUG=204049 TEST=None until ckb and ku are added to the A-L list in a Chrome CL. When they're, language and input settings will have ckb and ku listed with the display name in the current UI language (e.g. Englsih) and native display name. Review URL: https://codereview.chromium.org/805073003
-
- Dec 11, 2014
-
-
Jungshik Shin (jungshik at google) authored
README.chromium update and locid.patch were not included in the previous check-in (6bbe98e6 reviewed at https://codereview.chromium.org/796153002 ). BUG=403902 TEST=NONE TBR=phajdan.jr@chromium.org Review URL: https://codereview.chromium.org/799643002
-
Jungshik Shin (jungshik at google) authored
The root cause is described in http://bugs.icu-project.org/trac/ticket/11421 The original two line fix is from gguangle@gmail.com. It was momdified to the minimum necessary because only |baseName| should be reset. TBR=phajdan.jr@chromium.org BUG=403902 TEST=Speculative: On Windows with the display language set to one other than English (e.g. ko, pt-BR, zh-CN, etc; just setting the default locale to one of them does not work on Win 7 or later or using --lang=foo does not work), start Chrome and go to ftp://ftp.vim.org. Render process should not crash. Review URL: https://codereview.chromium.org/796153002
-
- Dec 10, 2014
-
-
Jungshik Shin (jungshik at google) authored
A follow-up patch to https://codereview.chromium.org/751333003 BUG=433866 TEST=See the bug (comment 0) R=mbarbella@chromium.org Review URL: https://codereview.chromium.org/777783004
-
- Dec 04, 2014
-
-
Jungshik Shin (jungshik at google) authored
Add codereview.settings with the default values so that accepting default values in 'git cl config' works. BUG=438401 TEST=NONE R=vadimsh@chromium.org Review URL: https://codereview.chromium.org/775153003
-
- Nov 26, 2014
-
-
jshin@chromium.org authored
BUG=433866 TEST=See the bug. R=mbarbella@google.com Review URL: https://codereview.chromium.org/751333003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@293126 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Nov 19, 2014
-
-
jshin@chromium.org authored
http://bugs.icu-project.org/trac/ticket/11371 BUG=432209 TEST=See bug (comment 0 and comment 10) R=markbrand@google.com Review URL: https://codereview.chromium.org/734213003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292996 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Nov 14, 2014
-
-
jshin@chromium.org authored
BUG=422824,430353 TEST=NONE TBR=mbarbella Review URL: https://codereview.chromium.org/719943004 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292944 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
The patches for the following upstream two bugs are cherry-picked: http://bugs.icu-project.org/trac/ticket/11369 http://bugs.icu-project.org/trac/ticket/11370 BUG=422824,430353 TEST=See the bugs. TBR=mbarbella Review URL: https://codereview.chromium.org/732743002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292943 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Oct 30, 2014
-
-
torne@chromium.org authored
We don't support using the system ICU on android any more (the library is not included in the NDK and so cannot be relied on). Remove the support for it in ICU gyp files, to eliminate remaining references to the deprecated "android_src" gyp variable. BUG= R=mkosiba@chromium.org Review URL: https://codereview.chromium.org/690603002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292738 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Oct 15, 2014
-
-
jshin@chromium.org authored
Some layout tests and chrome browser tests compare the encoding labels case-sensitively. Fixing them all takes a while. In the meantime, change the ICU alias table to use exactly what they expects (Big5). Also drop big5-hkscs from the alias for Big5 for now. They'll be unified in another CL. See the layout test failures for PS #2 at https://codereview.chromium.org/649413002 BUG=412053 TEST=browser_tests --gtest_filter=*ncoding* TEST=Layout tests : http/tests/misc/char-encod*, fast/encoding/* R=jsbell@chromium.org TBR=jsbell@chromium.org Review URL: https://codereview.chromium.org/654153002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292476 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Oct 13, 2014
-
-
jshin@chromium.org authored
1. Replace the current encoding alias list (heavily patched) with our own HTML5-specific alias list. It's mostly generated from encoding.json, which is in turn derived from the WHATWG Encoding living standard. The most notable difference is that UTF-32 entries are kept until bug 417850 is resolved. Two other differences are: a. Two aliases for iso-8859-8-i (logical and csiso88598i) are not listed. They're dealt with in Blink. b. Chinese (gb*, big5*) aliases are not yet aligned to the encoding spec pending our decision on the unification of Big5 / Big5-HKSCS and GBK / GB18030. 2. Replace all the single-byte mapping tables with what's automatically generated with scripts/single-byte-gen.sh that uses index-* files downloaded from the WHATWG spec site. This will fix the decoding (ToUnicode) of windows-874 and windows-1253 while removing a lot of fallback/spurrious mapping entries in encoding direction ('FromUnicode') in a number of encodings. 3. Regenerate the ICU binary data files for Linux/Mac/Android/Windows/CrOS. 4. Remove now obsolete noop-*ucm files used to make ISO-2022-CN* decoder to turn an empty string. They're not necessary any more because ISO-2022-CN* were made 'replacement' encodings in Blink and our version of ICU does not have any code for ISO-2022-CN* any more. This cuts down the data size by 15kB. On Android, there's virtually no change in the data size because the previous data file on Android accidentally had smaller locale data for nb and ms. BUG=412053 TEST=browser_tests --gtest_filter="*ncoding*" TEST=net_unittest --gtest_filter="*ilenameUtil*" TEST=base_unittests --gtest_filter="*Conv*" TEST=Blink: fast/encoding/* TEST=http://www.w3.org/International/tests/repository/encoding/indexes/results-indexes TEST=http://www.w3.org/International/tests/repository/encoding/indexes/results-aliases TEST=http://www.w3.org/International/tests/repository/run?manifest=encoding/indexes&test=windows-1253_test TEST=http://www.w3.org/International/tests/repository/run?manifest=encoding/indexes&test=windows-874_test R=jsbell@chromium.org Review URL: https://codereview.chromium.org/598383002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292447 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Oct 04, 2014
-
-
maruel@chromium.org authored
That's the new name, and the next swarming_client roll will refuse the old name. R=jshin@chromium.org BUG= Review URL: https://codereview.chromium.org/607413006 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292317 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Sep 25, 2014
-
-
jshin@chromium.org authored
word_ja.txt has not been checked in although the icu data file for Android (android/icudtl.dat) was built with the file. README.chromium about Android's brkitr patch was not updated when moving to ICU 52.1. It's changed to reflect what we do in ICU 52.1 (where the upstream copy does have cjdict for CJ word breaking). TBR=andrewhayden@chromium.org BUG=NONE TEST=NONE (there's no change affecting Chrome/Blink builds). Review URL: https://codereview.chromium.org/609493003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292144 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
UCONFIG_NO_NON_HTML5_CONVERTER was added earlier to our copy of ICU, but it was never set to 1. It's my oversight. 1. Turns UCON..CONVERTER on in icu.gyp to drop all the encodings not required by the Encoding spec. Dropped encodings include UTF-7, BOCU, SCSU, CESU, ISCII, ISO-2022-{KR, CN*}, HZ-GB, ISO-2022-JP's other than the original. 2. A lot more sections of the ICU converter code are excluded when it's set to 1 including the code for LMB (Lotus Multibyte) encodings and X11 compound text encoding (icu common). 3. The character encoding detections for encodings excluded are also disabled. (icu i18n) 4. ISO-2022-{KR, CN*} and HZ-GB can be dropped now because Blink treats them as replacement encoding. The corresponding alias entries from convertrs.txt are also removed. 5. ibm-874 was removed. We used to need it before Blink started, but not any more. We only need windows-874. 6. A mistaken in convertrs.txt was corrected : Big5-HKSCS was pointing to an old mapping table. 7. Per ICU upstream's suggestion, use '-html' suffix instead of '-html5' for the encoding tables derived from the WHATWG's encoding spec (ibm866, shift_jis and euc-jp). The static 64-bit release build of Chrome on Linux went down from 141,596,616 to 141,491,968 bytes (~ 100 kB reduction). Besides, the icu data size got smaller by ~ 19 kB ( 10,490,576 to 10,471,008 bytes). See http://bugs.icu-project.org/trac/ticket/11296 for an upstream bug I've filed on the issue. BUG=76328 TEST=browser_tests --gtest_filter="*ncoding*" TEST=net_unittest --gtest_filter="*ilenameUtil*" TEST=base_unittests --gtest_filter="*Conv*" TEST=Blink: fast/encoding/* TEST=With shared library build, the following has no match. nm libicuuc.so | egrep -i '(bocu|scsu|utf7|2022kr|2022cn|iscii)' nm libicui18n.so | egrep -i '(2022kr|2022cn|ibm42)' TEST=With static library build, the following has no match. nm chrome | egrep -i '(bocu|scsu|utf7|2022kr|2022cn|iscii|ibm42)' R=jsbell@chromium.org, mark@chromium.org Review URL: https://codereview.chromium.org/587833004 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292131 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Sep 17, 2014
-
-
jshin@chromium.org authored
Cherry-pick an upstream patch from http://bugs.icu-project.org/trac/ticket/11054 BUG=402297 TEST=See the comment 7 and comment 27 in the bug (after rolling DEPS with this CL). R=ckocagil@chromium.org TBR=ckocagil@chromium.org Review URL: https://codereview.chromium.org/566073003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@292003 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Sep 03, 2014
-
-
torne@chromium.org authored
ICU fails to gyp when run with GYP_DEFINES="android_webview_build=1 use_system_icu=0 use_system_stlport=1" which is a combination of settings we're trying to bring up temporarily as we migrate away from system libraries. It fails because it's not permitted to specify link_settings in a target_condition as the processing is too late. Remove the invalid link_settings, since we can't do this outside target_conditions as it's not valid to use -lgabi++ on the host build of ICU. The link dependency on gabi++ will have to be satisfied manually in the main libwebviewchromium target instead for this configuration. BUG=409851 R=mkosiba@chromium.org TBR=jshin@chromium.org Review URL: https://codereview.chromium.org/527193003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291781 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Sep 02, 2014
-
-
jshin@chromium.org authored
1. Timezone data files (4 of them) in source/data/misc to 2014f (the latest) to prepare for an upcoming Russian timezone change. 2. Add Shift_JIS converter compliant to the WHATWG encoding spec. 3. Update converters.txt and ucmlocal.mk accordingly 4. Update the pre-built data files for Linux/Mac/Android/Windows. (icudt.dll is not updated in this CL. It's not used in the default configuration. It'll be updated in a separate CL). 5. Fix a typo in ibm866_gen.sh. The acual table used does not need a change. BUG=277062,404445 TEST=After rolling icu to this revision, the following tests should pass. TEST=Blink: fast/encoding/* all pass except for fast/encoding/api/ascii-supersets.html that should fail by *passing* the test for Shift_JIS, which is expected to fail. Blink layout tests needs to be updated. TEST=browser_tests --gtest_filter="*ncoding*" TEST=In JS console, run the following to check if Europe/Moscow is 3 hrs ahead of UTC after Oct 26 and 4 hrs ahead before that and if Asia/Kamchatka remains 12 hrs ahead of UTC. nov1_2014_1500=new Date("11/01/2014 15:00Z") nov1_2014_1500.toLocaleString("en", {timeZone: "Europe/Moscow"}) nov1_2014_1500.toLocaleString("en", {timeZone: "UTC"}) nov1_2014_1500.toLocaleString("en", {timeZone: "Asia/Kamchatka"}) oct24_2014_1500=new Date("10/24/2014 15:00Z") oct24_2014_1500.toLocaleString("en", {timeZone: "Europe/Moscow"}) oct24_2014_1500.toLocaleString("en", {timeZone: "UTC"}) oct24_2014_1500.toLocaleString("en", {timeZone: "Asia/Kamchatka"}) TEST=net_unittest --gtest_filter="*ilenameUtil*" TEST=base_unittests --gtest_filter="*Conv*" R=jsbell@chromium.org Review URL: https://codereview.chromium.org/497543003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291774 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Aug 28, 2014
-
-
thakis@chromium.org authored
TBR=rnk@chromium.org Review URL: https://codereview.chromium.org/515263002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291706 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
thakis@chromium.org authored
while building icu. The warning isn't very serious, and these functions are supposed to trap. Since ICU's operator new/delete overrides are technically invalid, I filed http://bugs.icu-project.org/trac/ticket/11122 with ICU for this. BUG=82385 Review URL: https://codereview.chromium.org/508343002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291699 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Aug 26, 2014
-
-
jshin@chromium.org authored
i18n/winmfmt.* has a few warnings emitted when compiled with clang on Windows due to the following issue: ISO C++11 does not allow conversion from string literal to 'wchar_t *' [-Wwritable-strings] This CL from thakis@ takes care of it. An upstream bug was filed : http://bugs.icu-project.org/trac/ticket/11102 BUG=82385 TEST=Build on Windows with clang and the above warning is not emitted while building ICU. R=thakis@chromium.org Review URL: https://codereview.chromium.org/507153002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291660 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Aug 21, 2014
-
-
scottmg@chromium.org authored
It defaults to 1 everywhere except android webview, but it looks like the windows condition wasn't updated after the switch. TBR=jshin@chromium.org R=jam@chromium.org BUG=404809 Review URL: https://codereview.chromium.org/486103002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@291179 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Aug 01, 2014
-
-
thakis@chromium.org authored
Review URL: https://codereview.chromium.org/433183002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@287122 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Jul 24, 2014
-
-
jshin@chromium.org authored
NaCl has to be identified by '__native_client__'. Currently, we assume that NaCl has a signature of Linux (e.g. __linux___), but that is not the case. Add a new platform (U_PF_NATIVE_CLIENT) to platform.h and treat it as Linux-based. Upstream bug : http://bugs.icu-project.org/trac/ticket/11033 BUG=NONE TEST=NaCl builds fine when ICU is rolled to this revision. R=ncbray@chromium.org Review URL: https://codereview.chromium.org/419013002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@285387 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
hjd@chromium.org authored
The Android WebView's use of the system ICU needs to be updated. The current copy of ICU in external/icu4c is being replaced with a copy in external/icu/icu4c/source so we need to update the include paths referenced by third_party/icu/icu.gyp. BUG=395244 R=andrewhayden@chromium.org Committed: 285202 Review URL: https://codereview.chromium.org//412473002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@285220 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Jun 18, 2014
-
-
jshin@chromium.org authored
Taken from https://codereview.chromium.org/297293005 (the CL for icu 4.6.1) by jam BUG=132145 TEST=none R=jam@chromium.org Review URL: https://codereview.chromium.org/341653003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@277999 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Jun 17, 2014
-
-
jshin@chromium.org authored
When ICU was rolled from icu4.6 to icu52, libicudata_nacl was not rebuilt on some build bots leading to an undefined symbol linker error (icudt52_dat not found). By making icudata_nacl target explicitly depend on uvernum.h, we can force icudata_nacl target to be rebuilt when there's an ICU version change. (a work-around suggested by Sergey). A more fundamental fix is to change the way nacl targets determine its dependencies See https://codereview.chromium.org/336993003/ BUG=132145 TEST=After rolling icu to 52, incremental build will not complain about missing symbol (icudt52_dat). R=sergeyu@chromium.org Review URL: https://codereview.chromium.org/338063002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@277885 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Jun 10, 2014
-
-
jshin@chromium.org authored
Applying cjhopman's CL [1] to icu 52 BUG=351052,132145 TEST=For Android build, ninja does not complain about multiple rules generating icudtl.dat TBR=cjhopman@chromium.org [1] https://codereview.chromium.org/317373005/ Review URL: https://codereview.chromium.org/322903003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@275893 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- May 24, 2014
-
-
jshin@chromium.org authored
When the upstream took our patches for CJ segmentation, over 10k words were dropped. Some of them are pretty common and not having them led to a Blink layout test failure. Add several of them back to cjdict.txt. In addition, remove a patch that breaks line breaking around single/double quotation marks. Rebuild the data for Linux/Mac/Windows/Android. BUG=132145 TEST=Once rolled, layouttest:fast/text/international/cjk-segmentation.html and fast/hyphen-min-preferred-width.html pass. TBR=mark Review URL: https://codereview.chromium.org/292123005 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@272650 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- May 20, 2014
-
-
jshin@chromium.org authored
The original CL is by maruel (see https://codereview.chromium.org/263823013/). I'm applying his CL to icu52. Below is maruel's CL description. This is not needed anymore, src/tools/isolate_driver.py parses the ninja files to figure out the .so/.dylib/.dll needed automatically. TBR=maruel@chromium.org BUG=336439 Review URL: https://codereview.chromium.org/297763003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@271774 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- May 06, 2014
-
-
jshin@chromium.org authored
This is to update the Android data files (dat and assembly) after https://codereview.chromium.org/264973016 is checked in. For Android, only the following part matters: - Add Norweigian (nb) and Malay (ms) locale data that were not included by mistake. BUG=132145 TEST=When ICU is rolled, unit_tests:ExtensionL10* pass. Review URL: https://codereview.chromium.org/268833007 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@268566 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- May 05, 2014
-
-
jshin@chromium.org authored
I was too aggressive in trimming the data and dropped the display names for languages that Chromium needs (for non-UI languages that are in the A-L list). It's not my intention (the comment in trim_data.sh said one thing, but the code did another). Besides, add Norweigian (nb) and Malay (ms) locale data that were not included by mistake. Also update trim_data.sh script NOT to drop 'ALIAS' lines which are used to indicate that a given locale is an alias to another locale. That also required adding ro_MD.txt (null locale which mo.txt is aliased to). The above three adds about 110kB to the icu data (from 10.3MB to 10.4MB). Also update the pre-built icu data files for Linux, Mac and Windows. The Android data will be updated in a follow-up patch. BUG=132145 TEST=When ICU is rolled, unit_tests:ExtensionL10* pass. TBR=mark Review URL: https://codereview.chromium.org/264973016 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@268285 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Apr 30, 2014
-
-
jshin@chromium.org authored
dl_open and friends are only used by ICU plugins that we don't use. Nacl-related targets (with newlib) fail because dlfcn.h is missing. Setting U_ENABLE_DYLOAD to 0, we don't have to worry about dlfcn.h any more (it's not included) BUG=132145 TEST=After running build/gyp_chromium, build 'base_i18n_nacl' target. It should succeed without complaining about dlfcn.h. R=sergeyu@chromium.org Review URL: https://codereview.chromium.org/263763002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@267357 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
Newlib has _timezone instead of __timezone. pNaCl with newlib needs this change. BUG=132145 TEST=When rolled, NaCl (with newlib) can be compiled without an error. R=mark@chromium.org Review URL: https://codereview.chromium.org/266503003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@267335 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
Port the icu46 gyp file change to icu52. See https://codereview.chromium.org/212593002 BUG=346626,132145 R=torne@chromium.org TBR=torne@chromium.org Review URL: https://codereview.chromium.org/257303002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@267148 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Apr 29, 2014
-
-
jshin@chromium.org authored
- Add missing half-width kana entries (omitted by mistake) - Drop 'extra' decoding only mapping. See https://www.w3.org/Bugs/Public/show_bug.cgi?id=25266 - Regenerate icu data files (*dat and assembly source files) for Linux, Mac, Windows and Android. (they'll not be shown at codereview.chromium.org because they're too large). BUG=132145,78847 TEST=When ICU is rolled in, base_unittests --gtest_filter=*ICU* and layout tests R=jsbell@chromium.org Review URL: https://codereview.chromium.org/251203003 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@266919 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
umutex.h and wintz.c have two compile errors on Windows. The former issue was fixed in the upstream, but the latter is not. I reported the latter to the upstream (see README.chromium) BUG=132145 TEST=Can build on Windows with icu52. R=mark@chromium.org TBR=mark@chromium.org Review URL: https://codereview.chromium.org/256183002 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@266901 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Apr 28, 2014
-
-
jshin@chromium.org authored
1. Generate and add windows/icudt.dll with the procedure outlined in README.chromium. It uses a out-of-tree copy of the upstream ICU along with our custom-built icudtl.dat and a locally modified version of makedata.mak. We used to have a separate build/ directory for VS solution/project files to build icudtl.dll. Maintaining them is rather cumbersom now that we want to update our ICU (major version changes) more frequently. Note that icudt.dll is not used by default (icu_use_data_file_flag=1). The GN build still uses it by default and we should not break that build. 2. Add scripts/make_mac_assembly.sh to simplify the generation of the icu data assembly source file for Mac. 3. Update README.chromium accordingly. This CL was uploaded and reviewed at https://codereview.chromium.org/255943004/ Due to a malfunction at codereview.chromium.org, I'm landing this CL manually in two parts. This check-in is the 2nd part of the CL dealing with #2 and #3 above. BUG=132145 TEST=None until icu is rolled to this version. git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@266602 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
jshin@chromium.org authored
1. Generate and add windows/icudt.dll with the procedure outlined in README.chromium. It uses a out-of-tree copy of the upstream ICU along with our custom-built icudtl.dat and a locally modified version of makedata.mak. We used to have a separate build/ directory for VS solution/project files to build icudtl.dll. Maintaining them is rather cumbersom now that we want to update our ICU (major version changes) more frequently. Note that icudt.dll is not used by default (icu_use_data_file_flag=1). The GN build still uses it by default and we should not break that build. 2. Add scripts/make_mac_assembly.sh to simplify the generation of the icu data assembly source file for Mac. 3. Update README.chromium accordingly. This CL was uploaded and reviewed at https://codereview.chromium.org/255943004/ Due to a malfunction at codereview.chromium.org, I'm landing this CL manually. BUG=132145 TEST=None until icu is rolled to this version. git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@266601 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Apr 24, 2014
-
-
jshin@chromium.org authored
Somehow icupkg (used in generating icudtl.dll on Windows) complains that coll/ro_MD required for coll/ro is missing in icudtl.dat. Because ro_MD is ust an empty alias and it's not a size issue, I'm adding it to silence icupkg. (will check with the upstream about this unexpected dependencies). Note that I'm not updating the assembly source files and icudtl.dat for Android because it should not matter to them. BUG=132145 TEST=On Windows, makedata in allinone.snl does not complain. Review URL: https://codereview.chromium.org/255673005 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@265987 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-