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

Issue 838826 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 823362
Owner:
Last visit > 30 days ago
Closed: May 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 2
Type: Bug-Regression



Sign in to add a comment

Regression r532650: Transform applied to element inside clip-path is not working properly

Project Member Reported by r...@igalia.com, May 2 2018

Issue description


Check the attached example and hover the text.
The text should move 100px to the right, however it first moves down and left,
then do the transition to the right and moves up to the expected position.

This was working fine in Chromium 65, so it seems a regression introduced in 66.

Doing a bisect the list of commits was:
https://chromium.googlesource.com/chromium/src/+log/e34e01b1b0987e418bc22e3ef1cf2e4ecaead264..f96028345e3b1a869165a2bab1e7c894bfe467bf

So I guess the main suspect is:
https://chromium.googlesource.com/chromium/src/+/f89ae106e06d15200d783cb6950961ba999edebe

This works fine in Firefox.
 
regression-r532650.html
447 bytes View Download
Cc: -trchen@chromium.org
Owner: trchen@chromium.org
Status: Assigned (was: Untriaged)
Looks like an incorrect layer offset during the composited animation.
Works on trunk. Bisects to this fix: https://chromium.googlesource.com/chromium/src/+/10a4e34cc9207c2f6045ae5154cded6fed51f995

trchen@, is this a merge candidate?
Labels: -Pri-3 Pri-2
Mergedinto: 823362
Status: Duplicate (was: Assigned)
Is this seen on a real site? Per policy we only bring stability fix and critical correctness fix to stable, that's why I didn't merge to M66. That said, if regression is observed on some high traffic site, I think we should merge it in that case.
Labels: Needs-Feedback
NextAction: 2018-05-14
We need feedback for comment #4. rego@, please let us know how this bug was found.

Comment 6 by r...@igalia.com, May 2 2018

Labels: -Needs-Feedback
Sorry for not realizing it was fixed in trunk.
For some reason my 67 was not updated, and it was failing there too.
I see it working fine in 67, 68 and trunk now.

This is a real website, but not public yet as it's under development.
Dunno if there are other websites out there hitting this particular problem.
I think we should not merge to M-66 then. It will be there in M-67.
Labels: Target-67 OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows
NextAction: ----

Comment 9 by r...@igalia.com, May 3 2018

It seems other people are hitting this issue, but I guess that's not enough for merging in M66 yet:
https://stackoverflow.com/questions/50076578/clip-path-and-child-animation-in-chrome
We are also facing this problem. Live site: http://report.scout24.com/2017/home.html
When you click on Stories in the menu.
Please add comments over at https://bugs.chromium.org/p/chromium/issues/detail?id=823362

We are currently deciding whether to merge the fix into M-66. We are likely to do so but no final decision has been made.

Sign in to add a comment