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

Issue 821328 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: May 2018
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Non-Regression: Chrome//settings is seen instead of chrome://settings/privacy when clicked on settings link in Google Terms of Service overlay

Project Member Reported by kebalaji@chromium.org, Mar 13 2018

Issue description

Chrome Version:65.0.3325.159/10323.55.0 Beta channel Reks,Minnie
OS: Chrome OS

What steps will reproduce the problem?
(1)Sign-in to user>> Open Play store>>In, Google Terms of Service overlay, click on Settings and observe

Actual: When clicked on Settings, it's navigating to the Network section of settings i.e., it's navigating to chrome://settings instead of chrome://settings/privacy

Expected: When clicked on Settings, it should navigate to the Privacy and Security section of settings.

This is a Non-Regression issue seen from M-62

NOTE: Issue is seen on M63, M64, M65 and M67 also

@khmel: Please confirm the Issue


 
ActualPlaystoreSettings.webm
5.6 MB View Download
ExpectedPlaystoreSettings.webm
1.4 MB View Download
Project Member

Comment 1 by bugdroid1@chromium.org, May 16 2018

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

commit cc46beb506e982c506c8d11e8d9838480047ef04
Author: khmel@google.com <khmel@google.com>
Date: Wed May 16 21:19:41 2018

arc: Open privacy settings instead top-level settings.

Settings link in ARC++ OptIn UI is used in section about statistics
collection. It is more usable to user to open settings at privacy page
that contains relevant information.

Test: Manually
Bug:  821328 
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I8a4d9809618dedfdae7ca06e55680290ffb3eb5d
Reviewed-on: https://chromium-review.googlesource.com/1061623
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Yury Khmel <khmel@google.com>
Cr-Commit-Position: refs/heads/master@{#559293}
[modify] https://crrev.com/cc46beb506e982c506c8d11e8d9838480047ef04/chrome/browser/chromeos/arc/arc_support_host.cc
[modify] https://crrev.com/cc46beb506e982c506c8d11e8d9838480047ef04/chrome/browser/resources/chromeos/arc_support/background.js

Comment 2 by khmel@chromium.org, May 16 2018

Status: Fixed (was: Assigned)

Sign in to add a comment