New issue
Advanced search Search tips

Issue 804986 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Update strings for fine-grained timezone

Project Member Reported by wzang@chromium.org, Jan 23 2018

Issue description

The strings in the screenshot should be replaced with the ones in doc

https://docs.google.com/document/d/1T-5TupfCSvU2AWTnThMotDKkP_FJexpajCt7I3NvLs4/edit

Namely,

1) Set automatically using your general location (default)

=>  Use your IP address to determine location (default)

2) Set automatically using your accurate location

=>  Use Wi-Fi or mobile networks to determine location



+Tom, I noticed in the code base that there's another string:

Set automatically using your WiFi network

Should this be replace by something else as well?
 
IMG_20180123_112436.jpg
6.0 MB View Download

Comment 1 by wzang@chromium.org, Jan 25 2018

Do we still need this for 65?

Comment 2 by wzang@chromium.org, Jan 26 2018

Status: WontFix (was: Assigned)

Comment 3 by wzang@chromium.org, Jan 26 2018

Status: Started (was: WontFix)
For "Set automatically using your WiFi network", please use "Use only Wi-Fi to determine location" instead.
Project Member

Comment 5 by bugdroid1@chromium.org, Jan 27 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ec7b1be56aaf98872140bdeb74c1e18f0027c33e

commit ec7b1be56aaf98872140bdeb74c1e18f0027c33e
Author: Wenzhao Zang <wzang@chromium.org>
Date: Sat Jan 27 07:37:13 2018

cros: Update strings for fine-grained timezone

TBR=rockot@chromium.org

Bug:  804986 
Change-Id: I4b16892cfe7705d4a5d6dd82ccba8de176283ca2
Reviewed-on: https://chromium-review.googlesource.com/889647
Reviewed-by: Wenzhao (Colin) Zang <wzang@chromium.org>
Commit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#532189}
[modify] https://crrev.com/ec7b1be56aaf98872140bdeb74c1e18f0027c33e/chrome/app/settings_strings.grdp

Comment 6 by wzang@chromium.org, Jan 27 2018

Labels: Merge-Request-65
Status: Fixed (was: Started)

Comment 7 by wzang@chromium.org, Jan 27 2018

Labels: OS-Chrome
Project Member

Comment 8 by sheriffbot@chromium.org, Jan 28 2018

Labels: -Merge-Request-65 Merge-Review-65 Hotlist-Merge-Review
This bug requires manual review: There is .grd file changes and we are only 36 days from stable.
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), cmasso@(iOS), bhthompson@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Hotlist-Merge-Review -Merge-Review-65
We are only taking string changes in critical scenarios, I am not sure that this is critical at this point so perhaps this string change can wait until 66?

Sign in to add a comment