Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(625)

Unified Diff: third_party/WebKit/Source/core/paint/PaintLayer.h

Issue 2259493004: Fix Compositing of Opaque Scrolling Layers and Add Tests (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/paint/PaintLayer.h
diff --git a/third_party/WebKit/Source/core/paint/PaintLayer.h b/third_party/WebKit/Source/core/paint/PaintLayer.h
index d003945cf4f18d84301a29ab3fa3d2d1ac88a7ab..b7e75297fa1ce0f52cea514e3262d19b347ad72e 100644
--- a/third_party/WebKit/Source/core/paint/PaintLayer.h
+++ b/third_party/WebKit/Source/core/paint/PaintLayer.h
@@ -433,6 +433,12 @@ public:
CompositedLayerMapping* compositedLayerMapping() const;
GraphicsLayer* graphicsLayerBacking() const;
GraphicsLayer* graphicsLayerBackingForScrolling() const;
+ // Returns true for layers with scrollable overflow which have a background
+ // that can be painted into the composited scrolling contents layer (i.e.
+ // the background can scroll with the content). When the background is also
chrishtr 2016/09/02 01:19:38 "can be painted into the composited scrolling cont
Stephen Chennney 2016/09/06 21:06:21 Will do.
+ // opaque this allows us to composite the scroller even on low DPI as we can
+ // draw with subpixel anti-aliasing.
+ bool shouldPaintBackgroundOntoScrollingContentsLayer() const;
// NOTE: If you are using hasCompositedLayerMapping to determine the state of compositing for this layer,
// (and not just to do bookkeeping related to the mapping like, say, allocating or deallocating a mapping),
// then you may have incorrect logic. Use compositingState() instead.

Powered by Google App Engine
This is Rietveld 408576698