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

Issue 664116 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 652666
Owner:
Closed: Nov 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Regression:Unnecessary reload icon is seen in 'search settings' field of guest user.

Reported by jshan...@etouch.net, Nov 10 2016

Issue description

Chrome Version: 56.0.2914.3 (Official Build) b57184fe310b2dd87729c8b08e3e30dde2394195-refs/branch-heads/2914@{#7} 32/64-bit.
OS:Mac(10.11.6, 10.12.1), Windows (7,8,8.1,10), Linux (14.04 LTS)

What steps will reproduce the problem?
(1)Launch chrome and click on avatar icon, click on "guest"
(2)Navigate to chrome://md-settings/ click on 'search settings' and enter any letter for e.g. 'e' , observe search icon.
(3)Clear letter and click on page and again click on search field, observe the search icon.

Actual: Unnecessary reload icon is seen in 'search settings' field of guest user (i.e. after step 2 & 3)

Expected: Reload icon should not be seen in 'search settings field of guest user.

This is a regression issue broken in 'M54' and below is the manual regression range:
Good Build: 54.0.2809.0 
Bad Build: 54.0.2810.2 





 
Actual_Search.mp4
291 KB View Download
Expected_Search.mp4
310 KB View Download
Labels: Needs-Bisect
Status: Untriaged (was: Unconfirmed)

Comment 2 by hdodda@chromium.org, Nov 10 2016

Cc: hdodda@chromium.org
Labels: -Needs-Bisect hasbisect-per-revision
Owner: mahmadi@chromium.org
Status: Assigned (was: Untriaged)
Using the per-revision bisect providing the bisect results,
ood Build: 54.0.2809.0 (Revision:408012)
Bad Build: 54.0.2810.2 (Revision:408294)

You are probably looking for a change made after 408111 (known good), but no later than 408112 (first known bad).

CHANGELOG URL:

The script might not always return single CL as suspect as some perf builds might get missing due to failure.
  
https://chromium.googlesource.com/chromium/src/+log/6b16de9d3ef1b121411cfc63954408b51b28d945..369fb42299c568256ce9c8821967b9c413ea4151

From the CL above, assigning the issue to the concern owner 

@mahmadi - Could you please check whether this is caused with respect to your change, if not please help us in assigning it to the right owner.

Review-Url: https://codereview.chromium.org/2106103006

Thanks!
Cc: mahmadi@chromium.org
Owner: dpa...@chromium.org
Hi Demetrios, I thought you might know better what's causing this.

Comment 4 by dpa...@chromium.org, Nov 14 2016

Mergedinto: 652666
Status: Duplicate (was: Assigned)

Sign in to add a comment