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

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

Issue 2784963002: Optimize the pre-paint tree walk and ClipRects (Closed)
Patch Set: none Created 3 years, 9 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/PrePaintTreeWalk.h
diff --git a/third_party/WebKit/Source/core/paint/PrePaintTreeWalk.h b/third_party/WebKit/Source/core/paint/PrePaintTreeWalk.h
index c7d2274da3fc556e599b076131c69f717ebfe7c0..52ab471143e9be78727463ec75bd91b681186378 100644
--- a/third_party/WebKit/Source/core/paint/PrePaintTreeWalk.h
+++ b/third_party/WebKit/Source/core/paint/PrePaintTreeWalk.h
@@ -31,22 +31,23 @@ class PrePaintTreeWalk {
// Invalidates paint-layer painting optimizations, such as subsequence caching
// and empty paint phase optimizations if clips from the context have changed.
- void invalidatePaintLayerOptimizationsIfNeeded(const LayoutObject&,
- PrePaintTreeWalkContext&);
+ void ALWAYS_INLINE
+ invalidatePaintLayerOptimizationsIfNeeded(const LayoutObject&,
+ PrePaintTreeWalkContext&);
// Returns in |clipRect| the clip applied to children for the given
// contaiing block context + effect, in the space of ancestorState adjusted
// by ancestorPaintOffset. Sets hasClip to true if a clip was applied.
- void computeClipRectForContext(
+ void ALWAYS_INLINE computeClipRectForContext(
const PaintPropertyTreeBuilderContext::ContainingBlockContext&,
const EffectPaintPropertyNode*,
const PropertyTreeState& ancestorState,
const LayoutPoint& ancestorPaintOffset,
- bool& hasClip,
FloatClipRect&);
// Returns true if the tree walk should stop before processing this object.
- bool shouldEndWalkBefore(const LayoutObject&, const PrePaintTreeWalkContext&);
+ bool ALWAYS_INLINE shouldEndWalkBefore(const LayoutObject&,
+ const PrePaintTreeWalkContext&);
PaintPropertyTreeBuilder m_propertyTreeBuilder;
PaintInvalidator m_paintInvalidator;

Powered by Google App Engine
This is Rietveld 408576698