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

Issue 659488 link

Starred by 1 user
Status: Assigned
Owner:
est...@chromium.org
Cc:
nyerramilli@chromium.org
hablich@chromium.org
█ brajkumar@chromium.org
█ tkonch...@chromium.org
sky@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 2
Type: Bug-Regression



Sign in to add a comment

download shelf item buttons (malicious/dangerous items) have wrong z order during animation

Reported by dmascare...@etouch.net, Oct 26 2016 
Chrome Version:55.0.2883.28 eec10a413008022b45380e6d0b44a1ac0b2ea414-refs/branch-heads/2883@{#305}
OS: Windows (7,8,10), Linux (14.04 LTS)

What steps will reproduce the problem?
1. Launch chrome and navigate to http://parkerly.com/sb-tests/downloads/downloads.html
2. Click on 'foo.spl' link, 2-3 times and observe at download shelf.

Actual: Weird animation is observe at download shelf while clicking on 'foo.spl' link.
Expected: Animation should be proper.

This is regression issue, broken in 'M 55' and below is manual bisect:

Good build:55.0.2853.0
Bad build:55.0.2854.0

Note: Issue is not seen on Mac OS.
actual_download.mp4
530 KB View Download
Exp_download.mp4
527 KB View Download

Comment 1 by brajkumar@chromium.org, Oct 26 2016

Cc: brajkumar@chromium.org
Components: Blink>JavaScript
Labels: hasbisect-per-revision
Owner: hablich@chromium.org
Status: Assigned (was: Unconfirmed)
Bisect Information:
------------------------
You are probably looking for a change made after 417118 (known good), but no later than 417119 (first known bad).

CHANGELOG URL:
----------------
https://chromium.googlesource.com/chromium/src/+log/911199cb30f608636c79d47e50dca84735e3a08f..609dcc05df557f55041e60eb14b6441685645ab3

@Hablich - Unable to find the actual suspect from the above log, could you please check and help us in assigning this issue to the right owner.


Thanks!

Comment 2 by tkonch...@chromium.org, Oct 26 2016

Labels: ReleaseBlock-Stable
Adding RB label as this is a recent regression.

Comment 3 by hablich@chromium.org, Oct 26 2016

Cc: hablich@chromium.org
Owner: ----
Status: Untriaged (was: Assigned)
Uhm, I doubt V8 is the culprit here. What exactly is not working? I was looking at the videos and the text and have no clue what is not working. Can you please elaborate? How did you bisect?

Comment 4 by hablich@chromium.org, Oct 26 2016

Labels: Needs-Bisect

Comment 5 by gov...@chromium.org, Oct 26 2016 

**** Bulk edit -  please ignore if not applicable ****

A friendly reminder that M55 Stable is launch is coming soon! Your bug is labelled as Stable ReleaseBlock, pls make sure to land the fix and get it merged into the release branch ASAP so it gets enough baking time in Beta (before Stable promotion). Thank you!

Comment 6 by brajkumar@chromium.org, Oct 27 2016

Labels: -Needs-Bisect
Owner: est...@chromium.org
Status: Assigned (was: Untriaged)
Performed re-bisect and got the below result as the suspect for this issue.

Bisect Information:
------------------------
You are probably looking for a change made after 417121 (known good), but no later than 417122 (first known bad).

CHANGELOG URL:
----------------
https://chromium.googlesource.com/chromium/src/+log/617caed52b42647f938ea49ad9afed9256078198..e84ef9b0ce92cf74aaad872583e1e757dfb95ad3

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

@estade - 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.

Thanks!

Comment 7 by est...@chromium.org, Oct 27 2016

Labels: -Pri-1 -M-55 -ReleaseBlock-Stable M-56 Pri-2
Summary: download shelf item buttons (malicious/dangerous items) have wrong z order during animation (was: Regression: Weird animation is observed at download shelf while clicking on 'foo.spl' link.)
this is a regression, it is my fault, but I don't think it's worthy of RBS

Comment 8 by tkonch...@chromium.org, Nov 17 2016 

Still able to reproduce the issue on Linux 14.04 using chrome latest version 56.0.2923.0

 estade@, Could you please take a look

Comment 9 by tkonch...@chromium.org, Nov 25 2016 

Just to update:

Still able to reproduce the issue on win10 using chrome latest version 57.0.2931.0

estade@, Could you please take a look

Comment 10 by krajshree@chromium.org, Jan 10 2017 

Just to update, still able to reproduce this issue on win-10 using latest canary #57.0.2976.5.

estade@ - Could you please have a look into this issue.

Thanks...!!

Comment 11 by est...@chromium.org, Jan 10 2017

Cc: sky@chromium.org
I started on a general fix for this here[1] but we could not agree on a good solution. In lieu of that, it's yet another case where we'll need to add a layer to a parent view just so we can mask children. Scott, wdyt?

[1] https://codereview.chromium.org/2514303004/

Comment 12 by sky@chromium.org, Jan 10 2017 

Agreed.
►

Sign in to add a comment