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

Issue 434104: Merge 33118 - Pull in icu@32651:... (Closed)

Created:
11 years ago by jungshik at Google
Modified:
9 years, 7 months ago
Reviewers:
jungshik at Google
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

Merge 33118 - Pull in icu@32651: Remove invuca.icu table from the ICU data file again(see http://codereview.chromium.org/399089/show) It was added to make Chrome not crash with a webkit patch for bug 30437 (https://bugs.webkit.org/show_bug.cgi?id=30437) However, the webkit patch was rolled back so that we don't need invuca.icu for now. Removing it will cut down the data size by 230k (before compression and in the Chrome executable on Linux/Mac and the ICU data dll on Linux). This has to be added back when it becomes necessary again to fix the aforementioned webkit bug. See also http://crbug.com/28132 (a bug about ~ 4kB of space wasted by copyright statements) BUG=20406 TEST=All the webkit tests pass. Review URL: http://codereview.chromium.org/418019 TBR=jshin@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=33157

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
MM DEPS View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
jungshik at Google
11 years ago (2009-11-25 23:47:21 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698