Regression : 'Remove this person' overlay doesn't open up properly as it gets stuck from RHS while opening.
Reported by
rp...@etouch.net,
Jun 7 2018
|
|||||
Issue descriptionVersion: 68.0.3440.17 (Official Build)Revision 3f4388bf791dc0987401140638818b80d5a81616-refs/branch-heads/3440@{#225}(32/64-bit) OS: Windows (7,8,8.1,10),Linux (14.04 LTS),Mac OS X(10.12.6,10.13.1,10.13.6) Pre-condition : Add 4 persons from Manage people overlay. What steps will reproduce the problem? 1. Launch chrome,click on 'Avatar' icon and then click on 'Manage people' to open overlay 2. Now click on iron icon of 'Person 1' profile and observe 'Remove this person' overlay Actual: 'Remove this person' overlay doesn't open up properly as it gets stuck while opening from RHS Expected: 'Remove this person' overlay should open up properly This is regression issue, broken in ‘M 62’ and below is the bisect info : Good build: 62.0.3169.0 (Revision: 490188). Bad build: 62.0.3170.0 (Revision: 490562). You are probably looking for a change made after 490250 (known good), but no later than 490251 (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/a4bce1abf6150a5f5fc35846931608610a64c964..a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4 Suspect : https://chromium.googlesource.com/chromium/src/+/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4 From the CL above, assigning the issue to the concern owner @shend- 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. NOTE: This issue is also reproducible on M-67 Stable (build # 67.0.3396.79),M-68 Dev (build # 68.0.3440.15) and M-69 Canary (build # 69.0.3452.0).
,
Jun 8 2018
With response to comment #1: Re-bisected this issue on different machine and again getting same bisect info as mentioned in the issue. @shend : Kindly look into this issue and please help to reassign if issue is not caused by your change. Update : Actual : After clicking on three dot icon on Person profile, delay is seen while opening the menu list from RHS Expected : After clicking on three dot icon on Person profile, menu list should open properly. Note : Kindly refer attached screen_shot for reference.
,
Jun 8 2018
Thanks! Yes I can see the problem now. +mahmadi who is on the OWNERS file for md_user_manager, which I think is what this account picker window is called? @mahmadi, my change fixes a bug in Blink where `transition: all` didn't transition z-index (so now it does transition z-index). I think what is happening is that the md_user_manager CSS uses a `transition: all` somewhere and it is "smoothly" transitioning the z-index of the menu that pops up (the z-index of the action-box-menu is explicitly set to 6 I believe). Previously, we jumped directly to the final z-index value, which is incorrect (see crbug.com/746468 ). Do you know where this transition might be specified? I think explicitly listing out the properties that is being transitioned will fix this. I'll assign this to you as you're probably in best position to fix this. +dstockwell, do you know where else CSSAnimations::PropertiesForTransitionAll might be used? It's only for `transition: all` right?
,
Jun 8 2018
I no longer work on this feature. Assigning to dpapad@ for triaging and finding an owner.
,
Jun 8 2018
According to the bisect, this regression happened last year. Lowering priority. |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by shend@chromium.org
, Jun 8 2018Owner: rp...@etouch.net