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

Issue 193873002: Invalidate OS-specific datetime cache on configuration change notification (Closed)

Created:
6 years, 9 months ago by jamesr
Modified:
6 years, 9 months ago
Reviewers:
ulan
CC:
v8-dev
Base URL:
https://chromium.googlesource.com/external/v8.git@master
Visibility:
Public.

Description

Invalidate OS-specific datetime cache on configuration change notification When V8 is informed that the system's date time configuration has changed, it should also drop its OS-specific caches of time zone information R=ulan@chromium.org

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -1 line) Patch
M src/date.cc View 1 chunk +1 line, -0 lines 0 comments Download
M src/platform.h View 1 chunk +2 lines, -0 lines 0 comments Download
M src/platform-posix.cc View 1 chunk +4 lines, -0 lines 0 comments Download
M src/platform-win32.cc View 3 chunks +7 lines, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
jamesr
Ulan - would you mind reviewing this and helping me figure out how to land ...
6 years, 9 months ago (2014-03-11 01:02:18 UTC) #1
jamesr
6 years, 9 months ago (2014-03-11 01:30:12 UTC) #2
Hmm, trybots appear unable to apply this patch.  I tried creating this from my
git checkout inside chromium but may have got something wrong.

Closing in favor of https://codereview.chromium.org/193933002 which was created
from a read-only SVN checkout.  The trybots seem happier with it.

Powered by Google App Engine
This is Rietveld 408576698