Regression: Excessive checkerboarding or jank when scrolling
Reported by
pshi...@etouch.net,
Aug 31 2017
|
|||||||||||||||||||||
Issue descriptionDevice 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.
,
Aug 31 2017
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!
,
Aug 31 2017
I was able to repro with ToT on Nexus 6.
,
Sep 1 2017
If it regression on scrolling in Amazon app and has per-cl bisect. It needs to be addressed/investigated before Stable. Marking as RBS.
,
Sep 5 2017
,
Sep 5 2017
,
Sep 5 2017
,
Sep 5 2017
,
Sep 5 2017
Issue 760889 has been merged into this issue.
,
Sep 5 2017
,
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
,
Sep 6 2017
,
Sep 6 2017
Issue 762179 has been merged into this issue.
,
Sep 6 2017
Issue 762224 has been merged into this issue.
,
Sep 6 2017
Issue 762193 has been merged into this issue.
,
Sep 6 2017
Issue 762188 has been merged into this issue.
,
Sep 6 2017
Issue 762187 has been merged into this issue.
,
Sep 6 2017
,
Sep 6 2017
,
Sep 6 2017
Issue 762002 has been merged into this issue.
,
Sep 6 2017
Issue 761725 has been merged into this issue.
,
Sep 6 2017
+ 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.
,
Sep 6 2017
I should've mentioned that this affects Android (not just webview).
,
Sep 7 2017
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.
,
Sep 7 2017
Issue 762968 has been merged into this issue.
,
Sep 7 2017
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.
,
Sep 7 2017
,
Sep 7 2017
For what it's worth, the bug was already in Dev -- see Issue 761270 . Agree this should block Beta, not Dev.
,
Sep 7 2017
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
,
Sep 7 2017
Issue 761709 has been merged into this issue.
,
Sep 7 2017
Issue 763007 has been merged into this issue.
,
Sep 8 2017
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
,
Sep 8 2017
,
Sep 10 2017
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.
,
Sep 10 2017
,
Sep 11 2017
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!
,
Sep 11 2017
Issue 763716 has been merged into this issue.
,
Sep 12 2017
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.
,
Sep 13 2017
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
,
Sep 13 2017
Issue 764013 has been merged into this issue.
,
Sep 13 2017
Issue 762043 has been merged into this issue. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 Deleted