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

Unified Diff: ui/compositor/clip_transform_recorder.h

Issue 1423653005: Further plumb visual rect into cc:DisplayItemList. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix cc unit tests. Created 5 years, 1 month 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: ui/compositor/clip_transform_recorder.h
diff --git a/ui/compositor/clip_transform_recorder.h b/ui/compositor/clip_transform_recorder.h
index 276b845cf5b3351dd8036b7e0085b848a624fa5e..fa18360bf239cf152ee7cf2920b4c25e1ec0946b 100644
--- a/ui/compositor/clip_transform_recorder.h
+++ b/ui/compositor/clip_transform_recorder.h
@@ -9,6 +9,7 @@
#include "base/macros.h"
#include "ui/compositor/compositor_export.h"
+#include "ui/gfx/geometry/rect.h"
namespace cc {
class DisplayItem;
@@ -31,7 +32,10 @@ class PaintContext;
// be clipped/transformed.
class COMPOSITOR_EXPORT ClipTransformRecorder {
public:
- explicit ClipTransformRecorder(const PaintContext& context);
+ // |visual_rect| is expected to be in the coordinate space of the
danakj 2015/11/06 00:32:45 This belongs on visual_rect_, or should talk about
wkorman 2015/11/16 19:01:03 Renamed to layer_size.
+ // layer that contains the view.
+ explicit ClipTransformRecorder(const PaintContext& context,
+ const gfx::Size& visual_size);
~ClipTransformRecorder();
void ClipRect(const gfx::Rect& clip_rect);
@@ -46,6 +50,7 @@ class COMPOSITOR_EXPORT ClipTransformRecorder {
TRANSFORM,
};
const PaintContext& context_;
+ const gfx::Rect visual_rect_;
danakj 2015/11/06 00:32:45 would it make sense to call it layer_bounds_ or la
wkorman 2015/11/16 19:01:03 Renamed to layer_bounds_.
// If someone needs to do more than this many operations with a single
// ClipTransformRecorder then increase the size of the closers_ array.
Closer closers_[4];

Powered by Google App Engine
This is Rietveld 408576698