Fix paint checkerboarding over-reporting
We failed to update scrolling_contents_cull_rect on the cc scroll tree if a scroll only caused repaint but not a full PaintArtifactCompositor update. The wrong cull rect caused false paint checkerboarding. Now update scrolling_contents_cull_rect on the cc scroll tree during PaintChunksToCcLayer::UpdateLayerProperties() to ensure it is updated for both full and repaint-only updates. (cherry picked from commit 633b1b17) Bug: 373664061 Change-Id: Ibbc153e256f5f6c64c6003ad8e3a820eff60882a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5940118 Reviewed-by:Philip Rogers <pdr@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Original-Commit-Position: refs/heads/main@{#1372212} Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5962359 Auto-Submit: Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Cr-Commit-Position: refs/branch-heads/6778@{#782} Cr-Branched-From: b21671ca-refs/heads/main@{#1368529}
Showing
- third_party/blink/renderer/core/paint/compositing/compositing_test.cc 63 additions, 15 deletions...blink/renderer/core/paint/compositing/compositing_test.cc
- third_party/blink/renderer/platform/graphics/compositing/paint_chunks_to_cc_layer.cc 13 additions, 0 deletions...platform/graphics/compositing/paint_chunks_to_cc_layer.cc
- third_party/blink/renderer/platform/graphics/compositing/pending_layer.cc 3 additions, 2 deletions...k/renderer/platform/graphics/compositing/pending_layer.cc
- third_party/blink/renderer/platform/graphics/compositing/property_tree_manager.cc 1 addition, 7 deletions...er/platform/graphics/compositing/property_tree_manager.cc
Loading
Please register or sign in to comment