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

Issue 2243493003: arc: Fix logic of selection terms version in OptIn. (Closed)

Created:
4 years, 4 months ago by khmel
Modified:
4 years, 4 months ago
Reviewers:
CC:
chromium-reviews
Base URL:
https://chromium.googlesource.com/chromium/src.git@2785
Target Ref:
refs/pending/branch-heads/2785
Project:
chromium
Visibility:
Public.

Description

arc: Fix logic of selection terms version in OptIn. We are not allowed to show ToS in different than Chrome UI language, except the case when we are unable to determine right combination of language and zone and show ToS in default English. This also provides handling of navigator.language in 2 forwats, short and long which can appear in terms. BUG=636178 BUG=b/30434808 TEST=Manually on device, various combinations of time zone, UI languages and VPN. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation NOTRY=true NOPRESUBMIT=true TBR=reviewer@chromium.org Review-Url: https://codereview.chromium.org/2234573002 Cr-Commit-Position: refs/heads/master@{#411077} (cherry picked from commit b4cf059891075dc7f6e45bdc114118c1e5579f46)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+53 lines, -13 lines) Patch
M chrome/browser/resources/chromeos/arc_support/playstore.js View 1 chunk +53 lines, -13 lines 0 comments Download

Messages

Total messages: 6 (3 generated)
khmel
lgtm
4 years, 4 months ago (2016-08-11 17:40:23 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2243493003/1
4 years, 4 months ago (2016-08-11 17:40:54 UTC) #4
commit-bot: I haz the power
4 years, 4 months ago (2016-08-11 18:05:54 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1)

Powered by Google App Engine
This is Rietveld 408576698