- 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
-
- Sep 25, 2014
-
-
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 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
-
- 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 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
-
-
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 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
-
- 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
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 18, 2014
-
-
jshin@chromium.org authored
Add a shell script to trim the ICU data further : trim_data.sh along with locale list files. The script does the following: 1. Remove the display names of languages NOT listed in Chrome's Accept-Language list. (800kB) 2. Minimize the locale data for locales listed in the A-L list that are not a UI locale in Chrome. For those locales, exemplar characters, the display name in the native language and layout direction are included. (640kB) 3. Filter the region data to drop numeric region display names other than 419 (Latin-America). (50kB) 4. Filter the currency data (display name and plurals) for historic currencies. (200kB) This CL also checks in icudtl.dat (source/data/in) and icudt_dat.S (mac and linux). Note that I dropped '52' (the version number) in the assembly source file name and icu.gyp was adjusted accordingly. With all these changes, icudtl.dat is ~ 800kB larger than that in ICU 4.6. The 7z compression (as used by the installer) makes the size difference go down to ~ 130kB. BUG=132145 TEST=The icudtl.dat (uncompressed) is about 10.7MB instead of 12.4MB without this CL. R=mark@chromium.org Review URL: https://codereview.chromium.org/239543018 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@264811 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-
- Apr 01, 2014
-
-
jshin@chromium.org authored
Update icu.gyp and related files to list C/C++ source files and header files. Besides, HAVE_DLOPEN is set to 0 explicitly in icu.gyp. It's 0 by default in ICU 4.6.1 but is 1 by default in ICU 52.1, but we don't need it. (it's only necessary for ICU plugins that Chromium does not use). I also applied https://codereview.chromium.org/218153002/ (Nico's CL to simplify icu.gyp). BUG=132145 TEST=None R=mark@chromium.org, thakis@chromium.org Review URL: https://codereview.chromium.org/212623010 git-svn-id: http://src.chromium.org/svn/trunk/deps/third_party/icu52@260941 4ff67af0-8c30-449e-8e8b-ad334ec8d88c
-