diff options
author | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-01 23:02:38 +0000 |
---|---|---|
committer | danakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-11-01 23:02:38 +0000 |
commit | d0f98369e69763a2eeb7ea39097bf01ef234a39c (patch) | |
tree | 943e3e20a23e54d6de23d4007828a1d35c2bf6bf /cc/page_scale_animation.cc | |
parent | 1140046f0868117776b6df7e7c99c5bf3dd23731 (diff) | |
download | chromium_src-d0f98369e69763a2eeb7ea39097bf01ef234a39c.zip chromium_src-d0f98369e69763a2eeb7ea39097bf01ef234a39c.tar.gz chromium_src-d0f98369e69763a2eeb7ea39097bf01ef234a39c.tar.bz2 |
Remove most remaining references to IntRect and FloatRect.
The remaining uses are:
- Dealing with the output of Region::rects() which gives a vector of
WebCore::IntRects.
- Using FloatRect::isExpressibleAsIntRect.
Covered by existing tests; no new behaviour.
BUG=147395
R=enne
Review URL: https://codereview.chromium.org/11275113
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165542 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/page_scale_animation.cc')
-rw-r--r-- | cc/page_scale_animation.cc | 24 |
1 files changed, 15 insertions, 9 deletions
diff --git a/cc/page_scale_animation.cc b/cc/page_scale_animation.cc index 1bb02d7..59424d4 100644 --- a/cc/page_scale_animation.cc +++ b/cc/page_scale_animation.cc @@ -6,19 +6,22 @@ #include "cc/page_scale_animation.h" -#include "FloatRect.h" +#include "FloatPoint.h" #include "FloatSize.h" +#include "IntPoint.h" +#include "IntSize.h" +#include "ui/gfx/rect_f.h" #include <math.h> namespace cc { -scoped_ptr<PageScaleAnimation> PageScaleAnimation::create(const IntSize& scrollStart, float pageScaleStart, const IntSize& windowSize, const IntSize& contentSize, double startTime) +scoped_ptr<PageScaleAnimation> PageScaleAnimation::create(const IntSize& scrollStart, float pageScaleStart, const gfx::Size& windowSize, const gfx::Size& contentSize, double startTime) { return make_scoped_ptr(new PageScaleAnimation(scrollStart, pageScaleStart, windowSize, contentSize, startTime)); } -PageScaleAnimation::PageScaleAnimation(const IntSize& scrollStart, float pageScaleStart, const IntSize& windowSize, const IntSize& contentSize, double startTime) +PageScaleAnimation::PageScaleAnimation(const IntSize& scrollStart, float pageScaleStart, const gfx::Size& windowSize, const gfx::Size& contentSize, double startTime) : m_scrollStart(scrollStart) , m_pageScaleStart(pageScaleStart) , m_windowSize(windowSize) @@ -31,14 +34,18 @@ PageScaleAnimation::PageScaleAnimation(const IntSize& scrollStart, float pageSca { } +PageScaleAnimation::~PageScaleAnimation() +{ +} + void PageScaleAnimation::zoomTo(const IntSize& finalScroll, float finalPageScale, double duration) { if (m_pageScaleStart != finalPageScale) { // For uniform-looking zooming, infer the anchor (point that remains in // place throughout the zoom) from the start and end rects. - FloatRect startRect(IntPoint(m_scrollStart), m_windowSize); - FloatRect endRect(IntPoint(finalScroll), m_windowSize); - endRect.scale(m_pageScaleStart / finalPageScale); + gfx::RectF startRect(cc::FloatPoint(cc::IntPoint(m_scrollStart)), m_windowSize); + gfx::RectF endRect(cc::FloatPoint(cc::IntPoint(finalScroll)), m_windowSize); + endRect.Scale(m_pageScaleStart / finalPageScale); // The anchor is the point which is at the same ratio of the sides of // both startRect and endRect. For example, a zoom-in double-tap to a @@ -73,9 +80,8 @@ void PageScaleAnimation::zoomWithAnchor(const IntSize& anchor, float finalPageSc m_scrollEnd -= anchor; m_scrollEnd.clampNegativeToZero(); - FloatSize scaledContentSize(m_contentSize); - scaledContentSize.scale(finalPageScale / m_pageScaleStart); - IntSize maxScrollPosition = roundedIntSize(scaledContentSize - m_windowSize); + gfx::SizeF scaledContentSize = m_contentSize.Scale(finalPageScale / m_pageScaleStart); + IntSize maxScrollPosition = roundedIntSize(cc::FloatSize(scaledContentSize) - cc::IntSize(m_windowSize)); m_scrollEnd = m_scrollEnd.shrunkTo(maxScrollPosition); m_anchor = anchor; |