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

Issue 3598015: Fix the default timezone detection in ICU.... (Closed)

Created:
10 years, 2 months ago by jungshik at Google
Modified:
9 years, 7 months ago
Reviewers:
Evan Martin, evanm
CC:
chromium-reviews
Visibility:
Public.

Description

Fix the default timezone detection in ICU. This is the second patch for this issue. The previous patch was incomplete in that if a match was found for /etc/locatime in 'posix' directory under /usr/share/zoneinfo (say, posix/America/New_York) instead of at the top-level in /usr/share/zoneinfo (say, America/New_York), the DST offset is set to 0 while the raw-offset is set correctly. This leads the history and file:// to show the time stamp one hour behind the 'wall-clock time' when the DST is in effect. The upstream bug is http://bugs.icu-project.org/trac/ticket/8014 BUG=38632 TEST=1. Make a file locally and open the directory with the file when daylight saving time is in effect. The time stamp should match that of 'ls' command. 2. Visit a web page and open up the history tab and see the entry has the correct time stamp instead of time stamp 1-hr behind when daylight saving time is in effect. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=62048

Patch Set 1 #

Patch Set 2 : '' #

Patch Set 3 : better alternative / a small clean-up #

Patch Set 4 : '' #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+108 lines, -10 lines) Patch
M third_party/icu/README.chromium View 1 chunk +6 lines, -0 lines 0 comments Download
A third_party/icu/patches/linuxtz3.patch View 1 2 3 1 chunk +74 lines, -0 lines 1 comment Download
M third_party/icu/source/common/putil.c View 2 3 4 chunks +28 lines, -10 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
jungshik at Google
10 years, 2 months ago (2010-10-08 21:41:13 UTC) #1
Evan Martin
10 years, 2 months ago (2010-10-08 22:00:14 UTC) #2
The code LGTM, minor suggestion inside (maybe just for when the patch lands
upstream)

http://codereview.chromium.org/3598015/diff/9001/10002
File third_party/icu/patches/linuxtz3.patch (right):

http://codereview.chromium.org/3598015/diff/9001/10002#newcode22
third_party/icu/patches/linuxtz3.patch:22: +static void removeZoneIDPrefix(const
char** id) {
"skip" might be a better verb than "remove", as the latter indicates that it's
modifying the argument, but you're just advancing the pointer here.

Powered by Google App Engine
This is Rietveld 408576698