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

Issue 149953006: Use Android-specific icudata file (Closed)

Created:
6 years, 10 months ago by jungshik at Google
Modified:
6 years, 10 months ago
CC:
chromium-reviews, Xianzhu, andrewhayden, Torne, Miguel Garcia
Visibility:
Public.

Description

Use Android-specific icudata file Android build needs to use a separate icu data file. When icu_use_data_file_flag is on, it has to copy android/icudtl.dat. The Chrome-side of change is https://codereview.chromium.org/156333002/ BUG=72633 TEST=When rolled in in DEPS, Linux/Mac/Windows builds go as before (i.e. have third_party/icu/source/data/in/icudtl.dat copied to the top build directory). R=andrewhayden@chromium.org, mark@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=249466

Patch Set 1 #

Patch Set 2 : remove tabs #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -3 lines) Patch
M icu.gyp View 1 2 chunks +11 lines, -3 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jungshik at Google
6 years, 10 months ago (2014-02-06 08:25:02 UTC) #1
jungshik at Google
Mark, can you take a look? Thanks
6 years, 10 months ago (2014-02-06 08:31:21 UTC) #2
Mark Mentovai
LGTM
6 years, 10 months ago (2014-02-06 14:22:35 UTC) #3
Andrew Hayden (chromium.org)
LGTM +miguelg +torne as FYI
6 years, 10 months ago (2014-02-06 18:07:27 UTC) #4
jungshik at Google
6 years, 10 months ago (2014-02-06 18:57:05 UTC) #5
Message was sent while issue was closed.
Committed patchset #2 manually as r249466 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698