Chromium Code Reviews| Index: ui/compositor/paint_context.h |
| diff --git a/ui/compositor/paint_context.h b/ui/compositor/paint_context.h |
| index d63f11f7852b44c328b0cef7f855a0c0b1eef8de..25e1f60832c1fa4a01e037bcc300b2e131f8fba7 100644 |
| --- a/ui/compositor/paint_context.h |
| +++ b/ui/compositor/paint_context.h |
| @@ -56,15 +56,20 @@ class COMPOSITOR_EXPORT PaintContext { |
| return PaintContext(canvas_); |
| } |
| - // When true, IsRectInvalidated() can be called, otherwise its result would be |
| + // When true, IsRectInvalid() can be called, otherwise its result would be |
| // invalid. |
| - bool CanCheckInvalidated() const { return !invalidation_.IsEmpty(); } |
| + bool CanCheckInvalid() const { return !invalidation_.IsEmpty(); } |
| + |
| + // When true, if a thing is not invalidated it does not need to paint itself. |
| + // When false, everything should provide an output when painting regardless of |
| + // being invalidated in order to remain visible. |
| + bool EarlyOutOfPaintingWhenValid() const { return !!canvas_; } |
|
sky
2015/04/23 16:43:31
Nit: ShouldEarlyOut...?
danakj
2015/04/23 16:46:14
Ya!
|
| // When true, the |bounds| touches an invalidated area, so should be |
| // re-painted. When false, re-painting can be skipped. Bounds should be in |
| // the local space with offsets up to the painting root in the PaintContext. |
| - bool IsRectInvalidated(const gfx::Rect& bounds) const { |
| - DCHECK(CanCheckInvalidated()); |
| + bool IsRectInvalid(const gfx::Rect& bounds) const { |
| + DCHECK(CanCheckInvalid()); |
| return invalidation_.Intersects(bounds + offset_); |
| } |
| @@ -86,6 +91,9 @@ class COMPOSITOR_EXPORT PaintContext { |
| friend class ClipTransformRecorder; |
| friend class CompositingRecorder; |
| friend class PaintRecorder; |
| + // The Cache class also needs to access the DisplayItemList to append its |
| + // cache contents. |
| + friend class PaintCache; |
| PaintContext& operator=(const PaintContext& other) = delete; |