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

Unified Diff: cc/layer_impl.h

Issue 11503005: cc: Refactor content scale/bounds into draw properties (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix shadowing Created 8 years 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: cc/layer_impl.h
diff --git a/cc/layer_impl.h b/cc/layer_impl.h
index 889e1e11eba85ec828d7711fe586c53fec73b724..362182eeae46fe65edaace7153599463b976d690 100644
--- a/cc/layer_impl.h
+++ b/cc/layer_impl.h
@@ -190,14 +190,13 @@ public:
void setBounds(const gfx::Size&);
const gfx::Size& bounds() const { return m_bounds; }
- // ContentBounds may be [0, 1) pixels larger than bounds * contentsScale.
- // Don't calculate scale from it. Use contentsScale instead for accuracy.
void setContentBounds(const gfx::Size&);
- gfx::Size contentBounds() const { return m_contentBounds; }
+ gfx::Size contentBounds() const { return m_drawProperties.content_bounds; }
- float contentsScaleX() const { return m_contentsScaleX; }
- float contentsScaleY() const { return m_contentsScaleY; }
+ float contentsScaleX() const { return m_drawProperties.contents_scale_x; }
+ float contentsScaleY() const { return m_drawProperties.contents_scale_y; }
void setContentsScale(float contentsScaleX, float contentsScaleY);
+ virtual void updateContentsScale(float ideal_contents_scale) { }
gfx::Vector2d scrollOffset() const { return m_scrollOffset; }
void setScrollOffset(gfx::Vector2d);
@@ -323,9 +322,6 @@ private:
gfx::PointF m_anchorPoint;
float m_anchorPointZ;
gfx::Size m_bounds;
- gfx::Size m_contentBounds;
- float m_contentsScaleX;
- float m_contentsScaleY;
gfx::Vector2d m_scrollOffset;
bool m_scrollable;
bool m_shouldScrollOnMainThread;

Powered by Google App Engine
This is Rietveld 408576698