Flickering images at bottom of page |
|||
Issue descriptionWhen the following URL is viewed, flickering occurs at the bottom of the page during scrolling: https://www.google.com/search?client=ms-android-google&ei=HQR7W7rFAcrB0PEPmpawwAk&q=avengers+infinity+war&gs_ssp=eJzj4tbP1TcwNDJLssjOM2AEABghA04&oq=avengers+i+&gs_l=mobile-gws-wiz-serp.1.0.46i10j0j46j0j0i10.5015.5680..6352...0.0..0.84.221.3......0....1.........0i71j46i39j0i131i67j46i67j46i131i67j0i131i20i264j0i131j0i67.16Hff1Op6-s#wptab=s:H4sIAAAAAAAAAONgVuLWz9U3MDQyS7LIznvE6M4t8PLHPWEpu0lrTl5jtODiCs7IL3fNK8ksqRTS4mKDshS4-KWQ9WkwSPFyIQvw7GLicMvMyXVOLC4BADLpqfFkAAAA This doesn't appear to be a recent regression. Might be related to issue 806032, but that issue does seem to be fixed, so this might be something else.
,
Sep 7
ericrk: sorry to reassign your own reported bug back to you, but if this is p1, then it needs an owner. Can you find somebody to investigate?
,
Oct 30
Investigated this a bit: Previous fixes address cases where a tile is potentially drawn (part of a layer that draws), but the layer's rects did not yet take into account top bar changes. Unfortunately, they don't handle cases where we have earlier determined that an entire layer does not draw, and have marked the layer as not contributing to the drawn render surface: https://cs.chromium.org/chromium/src/cc/layers/picture_layer_impl.cc?rcl=5c8c1f9cd0a4059a130e5b3797ab28d97ae8a6a2&l=1674 In these cases we won't even process tiles from the layer and our other fixes won't take effect. It looks like we need to add a similar workaround somewhere earlier in our processing, maybe around here: https://cs.chromium.org/chromium/src/cc/trees/layer_tree_host_common.cc?rcl=5c8c1f9cd0a4059a130e5b3797ab28d97ae8a6a2&l=362 Marking P2 as this is a long-standing bug (not a regression), but we should still fix this if someone has bandwidth.
,
Nov 9
,
Nov 12
Root cause of this issue and 903689 seems different. 903689 can be fixed if we revert the patch ttps://chromium-review.googlesource.com/c/chromium/src/+/723982 But this issue still happening.
,
Nov 12
re #6, The patch per #6 shouldn't be reverted because it removed an experimental feature. The feature was skipping compositing small scrollers and with that we handled the small scrollable areas on main thread and no compositor involved. After removing this feature via the patch per #6, the scrollable area gets composited again and the related logic is handled on the compositor where the bug is really from, as ericrk@ mentioned per #3. |
|||
►
Sign in to add a comment |
|||
Comment 1 by zmo@chromium.org
, Aug 24