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

Regression: Excessive checkerboarding or jank when scrolling

Reported by pshi...@etouch.net, Aug 31 2017

Issue description

Device name: Pixel XL/N2G47O,Nexus 5X/N2G47O,Htc Desire 630/MMB29M,Samsung Galaxy J5 SM-J500F/MMB29M,Samsung Galaxy J2/LMY47X
WebView version: 62.0.3201.0
Application: Amazon Shopping
Application version: 12.7.0.300
Application package: in.amazon.mShop.android.shopping


Bisect Information:
Per-Version bisect information:
Good build: 62.0.3200.0
Bad build:  62.0.3201.0

Regression range: https://chromium.googlesource.com/chromium/src/+log/62.0.3200.0..62.0.3201.0?pretty=fuller&n=10000

Steps to reproduce:
(1)Launch Amazon app,Tap on 'Create an account'
(2)Tap on 'Country code' Dropdown list,Fast scroll up/down
(3)Observe

Actual result: After fast scrolling contents of the dropdown list appears temporarily blank. 

Expected result: The contents of the dropdown list should appear properly even after fast scrolling.

 

Comment 1 Deleted

Comment 2 by battun@chromium.org, Aug 31 2017

Labels: hasbisect-per-revision
Owner: sunn...@chromium.org
Status: Assigned (was: Untriaged)
Per-CL bisect information:
Good commit:498639
Bad commit:498640

Suspect CL:
https://chromium.googlesource.com/chromium/src/+/c7a7949c958b4867600ca716968da430fd9b952f

sunnyps@  Might be it looks like this issue is related to your change. please look into once, if its not related to your change please reassign to me. 
Thanks!
I was able to repro with ToT on Nexus 6. 
Labels: ReleaseBlock-Stable
If it regression on scrolling in Amazon app and has per-cl bisect. It needs to be addressed/investigated before Stable. Marking as RBS.


Components: Internals>GPU>Internals
Labels: OS-Chrome OS-Linux OS-Mac OS-Windows
Labels: -OS-Linux -OS-Windows -OS-Chrome -OS-Mac
Cc: sunn...@chromium.org piman@chromium.org
 Issue 761270  has been merged into this issue.
Cc: kkaluri@chromium.org
 Issue 761646  has been merged into this issue.
Cc: msrchandra@chromium.org ranjitkan@chromium.org rbasuvula@chromium.org nyerramilli@chromium.org
 Issue 760889  has been merged into this issue.
Cc: l...@chromium.org
 Issue 761593  has been merged into this issue.
Project Member

Comment 11 by bugdroid1@chromium.org, Sep 6 2017

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

commit 2eb3c8ac7ffc6840dc07f5e8312576e107eece23
Author: Sunny Sachanandani <sunnyps@chromium.org>
Date: Wed Sep 06 02:01:21 2017

cc: Flush after every tile task set completes.

Flushing in PrepareSendToParent isn't sufficient because that's skipped
when there's nothing to draw, and there can be nothing to draw if we
haven't flushed.

Flushing after all tiles are done is too late. The all tile set can be
very large for even simple pages.

Flushing after every task set and in PrepareSendToParent is reasonable
and seems to work well.

BUG= 760886 
R=piman

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I4915e8d10137d0c882c20464bce7563582691342
Reviewed-on: https://chromium-review.googlesource.com/651342
Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>
Cr-Commit-Position: refs/heads/master@{#499845}
[modify] https://crrev.com/2eb3c8ac7ffc6840dc07f5e8312576e107eece23/cc/resources/layer_tree_resource_provider.cc
[modify] https://crrev.com/2eb3c8ac7ffc6840dc07f5e8312576e107eece23/cc/tiles/tile_manager.cc
[modify] https://crrev.com/2eb3c8ac7ffc6840dc07f5e8312576e107eece23/cc/tiles/tile_manager.h

Cc: briander...@chromium.org
 Issue 762177  has been merged into this issue.
 Issue 762179  has been merged into this issue.
 Issue 762224  has been merged into this issue.
 Issue 762193  has been merged into this issue.
 Issue 762188  has been merged into this issue.
 Issue 762187  has been merged into this issue.

Comment 18 by sdy@chromium.org, Sep 6 2017

Labels: OS-Linux OS-Mac OS-Windows
Labels: Merge-Request-62
Cc: aelias@chromium.org ericrk@chromium.org
Issue 762002 has been merged into this issue.
Cc: tedc...@chromium.org
Issue 761725 has been merged into this issue.
Cc: amineer@chromium.org
+ amineer@

This bug is marked as RB-Stable but one of the bugs duped into this was RB-Dev. Are we doing a M62 dev release soon that we can merge the fix to? The fix landed on trunk yesterday isn't in a canary release yet.
I should've mentioned that this affects Android (not just webview).
Labels: TE-NeedsTriageFromMTV
Tested the issue on Windows 10 (Dell Precision M3800) using Chrome version M63 - 63.0.3208.0 as per the issue mentioned in original comment. Observed that issue is working as intended (Dropdown list doesn't appear to be blank). 

Issue is observed only on touch devices.

Adding the label 'TE-NeedsTriageFromMTV' to verify the issue on Mac and Linux touch devices.

Thanks.
 Issue 762968  has been merged into this issue.
Labels: -ReleaseBlock-Stable ReleaseBlock-Beta
I don't think this is a dev blocker - it's annoying but things still work.  That said it's a beta blocker, I don't think it meets our quality bar there.

We'll be shipping an M62 dev today cut from branch last night, so you missed that cut - but we'll be doing another next week before moving to beta so just make sure the fix is verified and then merged before EOD tomorrow and you should be OK.

Comment 27 by aluo@chromium.org, Sep 7 2017

Cc: bokan@chromium.org
 Issue 762465  has been merged into this issue.

Comment 28 by kbr@chromium.org, Sep 7 2017

For what it's worth, the bug was already in Dev -- see  Issue 761270 . Agree this should block Beta, not Dev.

Project Member

Comment 29 by sheriffbot@chromium.org, Sep 7 2017

Labels: -Merge-Request-62 Hotlist-Merge-Approved Merge-Approved-62
Your change meets the bar and is auto-approved for M62. Please go ahead and merge the CL to branch 3202 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Cc: yutak@chromium.org trchen@chromium.org pdr@chromium.org
 Issue 761709  has been merged into this issue.
 Issue 763007  has been merged into this issue.
Project Member

Comment 32 by bugdroid1@chromium.org, Sep 8 2017

Labels: -merge-approved-62 merge-merged-3202
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1024f08392734cf1a7557947b43d3b03e0f94b63

commit 1024f08392734cf1a7557947b43d3b03e0f94b63
Author: Sunny Sachanandani <sunnyps@chromium.org>
Date: Fri Sep 08 01:04:30 2017

[merge to M62] cc: Flush after every tile task set completes.

Flushing in PrepareSendToParent isn't sufficient because that's skipped
when there's nothing to draw, and there can be nothing to draw if we
haven't flushed.

Flushing after all tiles are done is too late. The all tile set can be
very large for even simple pages.

Flushing after every task set and in PrepareSendToParent is reasonable
and seems to work well.

BUG= 760886 
R=piman
TBR=sunnyps@chromium.org

(cherry picked from commit 2eb3c8ac7ffc6840dc07f5e8312576e107eece23)

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I4915e8d10137d0c882c20464bce7563582691342
Reviewed-on: https://chromium-review.googlesource.com/651342
Reviewed-by: Antoine Labour <piman@chromium.org>
Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#499845}
Reviewed-on: https://chromium-review.googlesource.com/656259
Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Cr-Commit-Position: refs/branch-heads/3202@{#79}
Cr-Branched-From: fa6a5d87adff761bc16afc5498c3f5944c1daa68-refs/heads/master@{#499098}
[modify] https://crrev.com/1024f08392734cf1a7557947b43d3b03e0f94b63/cc/resources/layer_tree_resource_provider.cc
[modify] https://crrev.com/1024f08392734cf1a7557947b43d3b03e0f94b63/cc/tiles/tile_manager.cc
[modify] https://crrev.com/1024f08392734cf1a7557947b43d3b03e0f94b63/cc/tiles/tile_manager.h

Status: Fixed (was: Assigned)
Cc: tdres...@chromium.org rbyers@chromium.org
Labels: -Restrict-View-Google
Summary: Regression: Excessive checkerboarding or jank when scrolling (was: After fast scrolling contents of the dropdown list appears temporarily blank in "Amazon Shopping" app. )
Updating summary to reflect the wide-ranging and serious impact of this bug - it basically destroys the scrolling user experience on most pages in Chrome.

Removing RVG and comment with google internal URLs - users/outside-developers will want to refer to this issue given it's broad scope and large impact (I'm sure many hours have already been wasted identifying this bug as the root cause of all the other duped issues for related symptoms).

Confirmed fixed in 62.0.3202.9.
Components: Blink>Scroll
Status: Verified (was: Fixed)
Fix merged to M62, Issue doesn't repro on latest M62- 62.0.3202.16 as per the steps mentioned in comment#0. And issue doesn't repro on latest M63: 63.0.3212.0

Tested on : Pixel XL/(NJH47D), Nexus 5X/(NRD90R), Samsung Galaxy J2/(LMY47X), Micromax Q372/5.1 (LRX21M), Sony Xperia XA/6.0(33.2.B.2.93), ASUS_Z010D /6.0.1(MMB29P), Htc Desire 630/6.0.1(1.00.400.3)

Thanks!
 Issue 763716  has been merged into this issue.
Tested the issue on Windows 10 (Dell Precision M3800) using Chrome version M62 - 62.0.3202.18 as per the issue mentioned in original comment. Observed that issue is working as intended (Dropdown list doesn't appear to be blank). 

As issue is observed only on touch devices requesting MTV team to verify the issue on Mac and Linux touch devices.

Thanks.
Verified fix on Nexus 5X/N2G47O with Chrome Canary "63.0.3213.0" and Chrome Dev M62.
Web sites visited:
https://en.m.wikipedia.org/wiki/Planet
https://en.m.wikipedia.org/wiki/Space_elevator

Comment 40 by bokan@chromium.org, Sep 13 2017

Issue 764013 has been merged into this issue.
Cc: sunyunjia@chromium.org ligim...@chromium.org gaschler@chromium.org sandeepkumars@chromium.org
 Issue 762043  has been merged into this issue.

Sign in to add a comment