Issue metadata
Sign in to add a comment
|
Regression: Flickering of header is observed on www.facebook.com
Reported by
dchau...@etouch.net,
May 31 2016
|
||||||||||||||||||||||
Issue descriptionChrome Version: 53.0.2753.0 (Official Build) bb0c85e1aaa882aa7c4c40b0e3da609573f07d8f-refs/heads/master@{#396745} 32/64-bit. OS: Windows(7,8,10), Linux (Ubuntu 14.04 LTS). What steps will reproduce the problem? 1. Launch chrome and login to www.facebook.com with valid credentials. 2. Click on 'Profile' icon at header of facebook (Navigates to profile section). 3. Now, Scroll up the page using mouse wheel and observe the header part of facebook. On scrooling, Header of facebook gets flicker. On scrooling, Header of facebook should not flicker. This is a regression issue,broken in M-53 series, below is bisect info. Good build: 53.0.2748.0 Bad build: 53.0.2750.0 Narrow bisect: https://chromium.googlesource.com/chromium/src/+log/44ac07c8316385250002371bec549fc6621826ec..2ab1696fd3a8d9e00f6736f7e5dba15f761452db?pretty=fuller&n=100 Suspecting: r396061 @ymalik: Kindly help to reassign, if your changes are not related to this issue. Note: 1. This issue is not seen on Mac OS. 2. To reproduce the issue frequently, repeat the step-3, 2-3 times. 3. This issue is not seen on other browsers like 'Firefox' and 'IE'. Kindly review the attached screen-cast for reference.
,
May 31 2016
Marking the above issue as RB-BETA as this is recent regression. Feel free to remove if not required. Thank you!
,
Jun 3 2016
,
Jun 3 2016
From the attached video, I think you're referring to the top nav changing in height and not flickering? Is this true? I can repro that on stable, so I don't think this is caused by r396061. It could be related to the takeover logic in some way though. I'll investigate further.
,
Jun 3 2016
,
Jun 6 2016
With response to comment #4: No, it's a flickering related issue only. i.e. on scrolling up the page using mouse wheel, header part of facebook gets flicker. Again done the bisect on different machine and found the same range as mentioned in bug. Note: Unable to capture the flickering in the video, but the same is reproducible on steps which is mentioned in the bug.
,
Jun 21 2016
,
Jun 23 2016
FYI, I don't see a "flicker" in the sense of the header disappearing and reappearing, but I do see the height change mentioned in #4, which appears to be a 1px shift when the scroll offset exceeds the height of the header.
,
Jun 27 2016
I was able to repro the flicker. For some background, when a main thread scrolling reason gets added on the main thread, we send a "takeover" message to the compositor thread, telling it to continue its scroll offset animation on the main thread. The problem is that there could be a delay between when the main thread scrolling reason gets added and when the takeover message is sent. As a result, we continue to animate on the compositor for longer than we should. Currently in Stable, we still see the flicker once in a while, but this got worse with r396061. https://codereview.chromium.org/2085873004/ brings it back to what's currently in stable. @dchaubey, can you verify this on the latest canary and close the issue if resolved?
,
Jun 28 2016
With response to comment #9: Retested the above issue on Windows-7 machine using latest canary: 53.0.2781.0 (Official Build). It's seems to be fixed and working as intended i.e. header of facebook doesn't flicker on scrolling up the page using mouse wheel.
,
Jul 7 2016
Moving this nonessential bug to the next milestone. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Jul 29 2016
Closing as per #10
,
Sep 27 2016
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by dchau...@etouch.net
, May 31 20161.0 MB
1.0 MB Download