New issue
Advanced search Search tips

Issue 710288 link

Starred by 1 user
Status: Fixed
Owner:
dpa...@chromium.org
Closed: Apr 2017
Cc:
dmazz...@chromium.org
dbeam@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

MD Settings: Should not be able to navigate to /manageAccessibility on non-CrOS

Project Member Reported by dpa...@chromium.org, Apr 11 2017 
Repro: Navigate to chrome://md-settings/manageAccessibility

Expected: This URL should not be a valid URL for non-CrOS (see [1]), it should forward user to chrome://md-settings

Actual: URL is accepted and lands the user at a mostly empty page (see screenshot).

[1] https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/a11y_page/a11y_page.js?l=35
url_exists_non_cros.png
19.0 KB View Download

Comment 1 by dpa...@chromium.org, Apr 11 2017

Labels: Hotlist-MD-Settings-Navigation

Comment 2 by dbeam@chromium.org, Apr 11 2017

Cc: dmazz...@chromium.org

Comment 3 by dpa...@chromium.org, Apr 11 2017

Owner: dpa...@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 4 by bugdroid1@chromium.org, Apr 11 2017 

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

commit ddcf763b75989af36bab3d5ace11e5ee5f3adead
Author: dpapad <dpapad@chromium.org>
Date: Tue Apr 11 04:26:16 2017

MD Settings: Remove /manageAccessibility route for non-CrOS

BUG= 710288 
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2808123003
Cr-Commit-Position: refs/heads/master@{#463534}

[modify] https://crrev.com/ddcf763b75989af36bab3d5ace11e5ee5f3adead/chrome/browser/resources/settings/route.js

Comment 5 by dbeam@chromium.org, Apr 11 2017

Status: Fixed (was: Started)
►

Sign in to add a comment