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

Issue 746468 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression
M61



Sign in to add a comment

transition does not interpolate z-index

Reported by wout.mer...@gmail.com, Jul 19 2017

Issue description

UserAgent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_4) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/59.0.3071.115 Safari/537.36

Steps to reproduce the problem:
1. Go to http://zomigi.com/demo/z-index_transition.html
2. hover over block One

What is the expected behavior?
Block One transitions smoothly over each other block separately

What went wrong?
Block One immediately gets the final z-index

Did this work before? N/A 

Does this work in other browsers? Yes

Chrome version: 59.0.3071.115  Channel: stable
OS Version: OS X 10.12.4
Flash Version: 

This does work with keyframe animation: https://codepen.io/chriscoyier/pen/gboJf

Both examples work fine in Safari
 
Labels: Needs-Bisect Needs-Triage-M61 OS-Windows
Able to reproduce this issue on Latest Canary#61.0.3161.0 for 'Win7'.
Cc: jmukthavaram@chromium.org ktyliu@chromium.org
Labels: -Type-Bug -Pri-2 -Needs-Bisect -Needs-Triage-M61 hasbisect-per-revision M61 OS-Linux Pri-1 Type-Bug-Regression
Owner: sashab@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on Mac 10.12.5, Windows 7 & Ubuntu 14.04 using chrome stable#59.0.3071.115 & Canary#61.0.3161.0 as per step 1.
Manual bisect info:
-------------------
Good-57.0.2983.0 -Revision-443819
Bad-57.0.2984.0 -Revision-443964

Per revision bisect info:
-------------------------
You are probably looking for a change made after 443844 (known good), but no later than 443845 (first known bad).
CHANGELOG URL:
The script might not always return single CL as suspectas some perf builds might get missing due to failure.
https://chromium.googlesource.com/chromium/src/+log/fd7c54b5c2775dc8d87c634e402b82d83aca2da1..be6d3f418dca5f76954e3e04108364fb6ce3be29

Possible suspect:
-----------------
https://chromium.googlesource.com/chromium/src/+/be6d3f418dca5f76954e3e04108364fb6ce3be29

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

sashab@Please take a look and reassign to the right owner if it is not related to your change.

CC'ing reviewer ktyliu@,as sashab Last visit shown as 22 days ago.
Thank you..!!



Cc: ligim...@chromium.org

Comment 4 by nainar@chromium.org, Jul 21 2017

Labels: Update-Quarterly

Comment 5 by shend@chromium.org, Jul 24 2017

Cc: sashab@chromium.org shend@chromium.org
Owner: ----
Status: Available (was: Assigned)

Comment 6 by shend@chromium.org, Jul 24 2017

Status: Started (was: Available)

Comment 7 by shend@chromium.org, Jul 24 2017

Owner: shend@chromium.org
Project Member

Comment 8 by bugdroid1@chromium.org, Jul 28 2017

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

commit a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4
Author: Darren Shen <shend@chromium.org>
Date: Fri Jul 28 03:56:07 2017

Fix off-by-one error in CSSAnimations::PropertiesForTransitionAll.

This patch fixes an off-by-one error in CSSAnimations which caused
the last CSS properties (in order of CSSPropertyID) to not transition
property (z-index was the last CSS property).

Bug:  746468 
Change-Id: I338996e4f8ac127cd2c6589f9f7ebedcf5d0e754
Reviewed-on: https://chromium-review.googlesource.com/582258
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: dstockwell <dstockwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#490251}
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/border-image-width-interpolation-expected.txt
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/list-style-image-interpolation-expected.txt
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/resources/interpolation-test.js
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/rotate-interpolation-expected.txt
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/webkit-mask-image-interpolation-expected.txt
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/LayoutTests/animations/interpolation/z-index-interpolation-expected.txt
[modify] https://crrev.com/a3c8a1b29d4800a78f7df20d61471a9ba6d4bbe4/third_party/WebKit/Source/core/animation/css/CSSAnimations.cpp

Comment 9 by shend@chromium.org, Jul 28 2017

Status: Fixed (was: Started)

Sign in to add a comment