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

Issue 629598 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug

Blocking:
issue 608535



Sign in to add a comment

MD Settings: Searching navigates to chrome://md-settings/quickUnlock/authenticate on ToT

Project Member Reported by michae...@chromium.org, Jul 19 2016

Issue description

1. Open chrome://md-settings
2. Type something in search, e.g. "a"

Page displays quick unlock section and navigates to chrome://md-settings/quickUnlock/authenticate.

Guessing this has something to do with quick_unlock_routing_behavior.js.
 

Comment 1 by dpa...@chromium.org, Jul 19 2016

Labels: Proj-MaterialDesign-WebUI

Comment 2 by dpa...@chromium.org, Jul 19 2016

Owner: tommycli@chromium.org
@tommycli: Could you take a look at this bug? One (or more) of the settings-subpages instances at https://cs.chromium.org/search/?q=people_page.html+dom-if%5C%22+name%3D&sq=package:chromium&type=cs, are modifying the current route when they are forced-rendered. This does not happen with other <settings-subpages> and it is a problem for searching, since force-rendering should not be modifiying the current route.
Cc: tommycli@chromium.org
Owner: jdufault@chromium.org
jdufault: I'm guessing this has to do with the redirect-to-authenticate behavior of Quick unlock. How can we make this work with search correctly?
Status: Started (was: Untriaged)
https://codereview.chromium.org/2160423002
Project Member

Comment 5 by bugdroid1@chromium.org, Jul 20 2016

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

commit 611982bedb154c8f6138d7bb881047520940919a
Author: jdufault <jdufault@chromium.org>
Date: Wed Jul 20 18:05:10 2016

Fix search always navigating to quick unlock authenticate screen.

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

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

[modify] https://crrev.com/611982bedb154c8f6138d7bb881047520940919a/chrome/browser/resources/settings/people_page/quick_unlock_choose_method.js
[modify] https://crrev.com/611982bedb154c8f6138d7bb881047520940919a/chrome/browser/resources/settings/people_page/quick_unlock_setup_pin.js

Status: Fixed (was: Started)
Labels: VerifyIn-54

Comment 8 by dchan@chromium.org, Oct 7 2016

Labels: VerifyIn-55
Status: Verified (was: Fixed)
Verified on ChromeOS 8872.18.0, 55.0.2883.20

Sign in to add a comment