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

Issue 687889 link

Starred by 1 user
Status: Fixed
Owner:
hcarmona@chromium.org
Closed: Feb 2017
Cc:
nyerramilli@chromium.org
kkaluri@chromium.org
dbeam@chromium.org
█ msrchandra@chromium.org
█ ranjitkan@chromium.org
dpa...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression

Blocking:
issue 671375



Sign in to add a comment

Regression : 'Advanced' section does not collapse at the second instance.

Reported by yfulgaon...@etouch.net, Feb 2 2017 
Chrome Version : 58.0.2999.5 (Official Build) 6c86d19d495b8ab045d74b3c19addca0fc1ffd48-refs/branch-heads/2999@{#5} 32/64 bit
OS : Windows(7,8,10), Mac(10.12.1, 10.11.6, 10.12), Linux 14.04

What steps will reproduce the problem?
1. Launch Chrome, navigate to chrome://md-settings, scroll down the page and click on 'Advanced'.
2. Open 'Main menu' side bar, click on 'Advanced' and hit 'Esc' key (side bar closes).
3. Again click on 'Advanced' button (seen on the page), open 'Main menu' side bar and click on 'Advanced'.
4. Observe.

Actual : 'Advanced' section does not collapse after clicking on it for the second instance.
Expected : 'Advanced' section should collapse at the second instance.

This is a regression issue broken in ‘M-56’, below is the Manual Regression range and will soon update other info.
Good build : 56.0.2891.0
Bad build : 56.0.2894.0
Actual_md-settings.mp4
768 KB View Download
Expected_md-settings.mp4
935 KB View Download

Comment 1 by kkaluri@chromium.org, Feb 2 2017

Cc: kkaluri@chromium.org
Labels: hasbisect-per-revision
Owner: dbeam@chromium.org
Status: Assigned (was: Unconfirmed)
Bisect Info:
===========
Good build : 56.0.2891.0,  Revision Range -425529
Bad build  : 56.0.2894.0,  Revision Range -425838

After executing the per-revision bisect script , i got the following CL's between good and bad build versions
===========================================
https://chromium.googlesource.com/chromium/src/+log/30caafb88a220a9352c7940d9d8b14070ef5d1f5..5f9bf9e2c68a25b9bd02639c70d31c9c627cb458

The suspecting Change Log is :
-----------
https://chromium.googlesource.com/chromium/src/+/5f9bf9e2c68a25b9bd02639c70d31c9c627cb458

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

dbeam@- Could you please look into this issue, if it's related to your change?  if not could you please help us to reassign this issue to the right owner.

Comment 2 by dbeam@chromium.org, Feb 2 2017

Blocking: 671375
Labels: -Pri-2 Hotlist-MD-Settings-Navigation Pri-1

Comment 3 by dbeam@chromium.org, Feb 11 2017

Cc: dbeam@chromium.org
Owner: hcarmona@chromium.org
this an issue with paper-submenu which would be solved by https://github.com/PolymerElements/paper-menu/pull/107

but should also be solved by just dropping <paper-menu>/<paper-submenu> like hcarmona@ is doing here:
https://codereview.chromium.org/2651293003/

Comment 4 by hcarmona@chromium.org, Feb 13 2017 

Yes, just tested on my machine. The changes from my CL will fix this issue.

Comment 5 by hcarmona@chromium.org, Feb 16 2017

Status: Fixed (was: Assigned)
Now that http://crrev.com/2651293003 has landed, this should also be fixed.
►

Sign in to add a comment