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

Issue 847046 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows
Pri: 1
Type: Bug-Regression



Sign in to add a comment

scrolling over transformed elements jumps around

Reported by bigchees...@gmail.com, May 27 2018

Issue description

UserAgent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/66.0.3359.139 Safari/537.36

Steps to reproduce the problem:
1. open attached file
2. Use scroll wheel while mouse pointer is over card images.

What is the expected behavior?
Smooth scrolling

What went wrong?
Regardless of the scroll direction, it keeps jumping down the scroll area.

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: 66.0.3359.139  Channel: n/a
OS Version: 10.0
Flash Version: 

This may only occur on Windows.  This doesn't happen with Firefox on Windows, and doesn't happen with Safari or Chrome on Mac OS X.  I was unable to test on Linux.

This only happens when the transform on hover is enabled.
 
testcase.html
281 KB View Download
Labels: Needs-Triage-M66
Components: Blink>Scroll
Cc: manoranj...@chromium.org
Labels: -Type-Bug -Pri-2 hasbisect-per-revision ReleaseBlock-Stable Triaged-ET M-67 Target-67 FoundIn-67 OS-Linux Pri-1 Type-Bug-Regression
Owner: jfernan...@igalia.com
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on reported version 66.0.3359.139, latest stable# 65.0.3325.181 and on latest Beta# 67.0.3396.56, the same is not seen on latest canary 69.0.3443.0 using Windows 10 and Ubuntu 14.04, hence providing reverse bisect info
Note: Issue is not seen on Mac 10.12.6
Reverse Bisect Info:
================
Last Bad build: 68.0.3423.0
First Good build: 68.0.3424.0

You are probably looking for a change made after 556468 (known good), but no later than 556469 (first known bad).
https://chromium.googlesource.com/chromium/src/+log/18502e69bd4080084bdf69c2cbeda8a629ab72a8..87f271283cfb054062dd24bd1e5fc4f768fca91b
Reviewed-on: https://chromium-review.googlesource.com/1027882

@Javier Fernandez: Please confirm the issue and help in re-assigning if it is not related to your change, please help in merging it to M-67 if applicable
Adding ReleaseBlock-Stable for M-67, feel free to remove it if not applicable.

Thanks!

Comment 4 by gov...@chromium.org, May 29 2018

Labels: -M-67 -Target-67 M-68 M-69 Target-69 Target-68
As this issue exists on M66 per comment #0 and #1, we won't block M67 stable for this. And seems like this is already fix in M68/M69. 
I can not confirm the fix is due to my patch in https://chromium-review.googlesource.com/1027882 though.
Umm, I was able to verify the issue in linux at r556468, and it's fixed by my patch in r556469. However, I still don't fully understand which was the root cause of the issue and why the patch identified by the bisect fixed it. 

My theory is that avoiding layouts during intrinsic size computation reduce the chances to be impacted by bugs related to repeated executions of the grid track sizing algorithm. We were aware of these issues long time ago and we indeed fixed many of them, but I think the mentioned CL is a big step forward the definitive solution. 

However, unless this issue's severity is considered high, I'd rather wait to see if the CL doesn't cause any regression before it's merged in the stable channel. 
 jfernandez@

Still seeing the same issue on Windows 10 & Linux. Please take a look into it as it is marked as stable blocker.
Thanks..!
umm, what do you want me to loot at ? per comment #c6 the patch in r556469 does indeed fix the issue, but it's a risky patch, so I'm not sure it's worth to merge in stable. 

I wouldn't oppose, though, if anybody consider the issue critical enough to take the risk.
BTW, the issue still has the Bug-Regression flag, which I think it's not accurate. 
Cc: abdulsyed@chromium.org gov...@chromium.org
M67 stable has been out since 05/29 and we're not planning any further M67 release unless extremely critical issue arise.

Pls request a merge to M68 if needed. Thank you.
Bulk update: M68 stable cut is scheduled for July 19th. This issue is marked as RB-Stable, so please take a look at it before. Thanks!
Labels: -ReleaseBlock-Stable
Per c#8, removing blocker label. Please feel free to re-add if someone thinks we need this critical merge to M68.

Sign in to add a comment