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

Issue 2025010: Add a separate icu data file for ChromeOS (Closed)

Created:
10 years, 7 months ago by jungshik at Google
Modified:
9 years, 7 months ago
Reviewers:
Mark Mentovai
CC:
chromium-reviews, satorux1
Visibility:
Public.

Description

Replace the full locale data for the following locales with the abridged version for ChromeOS: bn,gu,kn,ml,mr,or,ta,te,am,sw ChromeOS is not localized to them so that it does not need the full locale data. BUG=http://crosbug.com/2426 TEST=See the bug. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=47106

Patch Set 1 #

Patch Set 2 : '' #

Unified diffs Side-by-side diffs Delta from patch set Stats (+265 lines, -1 line) Patch
M third_party/icu/README.chromium View 1 1 chunk +6 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/am.txt View 1 chunk +23 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/bn.txt View 1 chunk +27 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/gu.txt View 1 chunk +24 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/kn.txt View 1 chunk +27 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/ml.txt View 1 chunk +27 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/mr.txt View 1 chunk +24 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/or.txt View 1 chunk +27 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/sw.txt View 1 chunk +24 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/ta.txt View 1 chunk +24 lines, -0 lines 0 comments Download
A third_party/icu/cros/locales/te.txt View 1 chunk +24 lines, -0 lines 0 comments Download
M third_party/icu/icu.gyp View 2 chunks +8 lines, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
jungshik at Google
10 years, 7 months ago (2010-05-13 00:50:31 UTC) #1
jungshik at Google
This is better than http://codereview.chromium.org/2040010
10 years, 7 months ago (2010-05-13 00:51:50 UTC) #2
jungshik at Google
cros/icudt42l_dat.s is a part of this CL, but it's not uploaded because it's too large. ...
10 years, 7 months ago (2010-05-13 00:59:19 UTC) #3
Mark Mentovai
10 years, 7 months ago (2010-05-13 01:10:04 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698