New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 757930 link

Starred by 1 user

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Consider implementing tzset workaround for Android < O

Project Member Reported by paulmiller@chromium.org, Aug 22 2017

Issue description

See https://android-review.googlesource.com/#/c/platform/bionic/+/321708/ where localtime_r is made to call tzset.

See internal bug 62563020, where taruhi@ recommends adding this workaround to v8/src/base/platform/platform-posix-time.cc on Android versions lacking the patch.
 
Cc: littledan@chromium.org yangguo@chromium.org jgruber@chromium.org
Labels: -Pri-3 Pri-2
Status: Available (was: Untriaged)
Cc: js...@chromium.org
V8 currently doesn't seem to call tzset on Linux at all. It might require updating the Sandbox to make such a call possible in Chrome [1]. Probably also a good idea to make sure to do this tzset call only on the affected Android platforms, as going through the sandbox has some cost.

[1] https://cs.chromium.org/chromium/src/content/zygote/zygote_main_linux.cc
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 23

Labels: Hotlist-Recharge-Cold
Status: Untriaged (was: Available)
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue.

Sorry for the inconvenience if the bug really should have been left as Available.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Owner: rmcilroy@chromium.org
Status: Assigned (was: Untriaged)
rmcilroy@ can you please evaluate?
This has been untouched for a year and is not on my radar. Is this still a priority? Technically it seems a bit complex due to sandbox concerns, so we would have to budget the work.
I don't think the priority has changed, although it was never high priority. It'll become obsolete in maybe 5 years when Android >= O reaches saturation :P

Sign in to add a comment