summaryrefslogtreecommitdiffstats
path: root/cc/thread_proxy.h
diff options
context:
space:
mode:
authordanakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-11-05 16:56:49 +0000
committerdanakj@chromium.org <danakj@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-11-05 16:56:49 +0000
commitec454716d4056ddc796788b0cab45ec99b16113b (patch)
treec12de89ddf61012d1d5eb7f8c8573773a9215f5a /cc/thread_proxy.h
parent992359d7cb13821cbc9cbf76d3e870860dea760d (diff)
downloadchromium_src-ec454716d4056ddc796788b0cab45ec99b16113b.zip
chromium_src-ec454716d4056ddc796788b0cab45ec99b16113b.tar.gz
chromium_src-ec454716d4056ddc796788b0cab45ec99b16113b.tar.bz2
cc: Remove all remaining use of WebCore Rect/Point/Size types from the compositor.
This change removes all IntPoint/FloatRect/IntSize/etc from the compositor. There remains an indirect dependency on these types through the WebCore::Region class, which we wrap but need to replace. However, the wrapper there hides the WebCore types inside it, so there are now no references to the types from anywhere else in the compositor. I went back and forth on how to deal with scroll "positions". The name suggested that they should be Points, and that the deltas should be Vectors. However this lent itself to super awkward math at times. In the end, it was much cleaner to make all scroll "positions" into scroll "offsets" and represent everything as Vectors. Covered by existing tests; no change in behaviour. R=enne BUG=147395 Review URL: https://codereview.chromium.org/11367080 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@165947 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'cc/thread_proxy.h')
-rw-r--r--cc/thread_proxy.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/cc/thread_proxy.h b/cc/thread_proxy.h
index 1a24ec5..7e14347 100644
--- a/cc/thread_proxy.h
+++ b/cc/thread_proxy.h
@@ -30,7 +30,7 @@ public:
// Proxy implementation
virtual bool compositeAndReadback(void *pixels, const gfx::Rect&) OVERRIDE;
- virtual void startPageScaleAnimation(const IntSize& targetPosition, bool useAnchor, float scale, base::TimeDelta duration) OVERRIDE;
+ virtual void startPageScaleAnimation(gfx::Vector2d targetOffset, bool useAnchor, float scale, base::TimeDelta duration) OVERRIDE;
virtual void finishAllRendering() OVERRIDE;
virtual bool isStarted() const OVERRIDE;
virtual bool initializeContext() OVERRIDE;
@@ -111,7 +111,7 @@ private:
void beginFrameCompleteOnImplThread(CompletionEvent*, ResourceUpdateQueue*);
void beginFrameAbortedOnImplThread();
void requestReadbackOnImplThread(ReadbackRequest*);
- void requestStartPageScaleAnimationOnImplThread(IntSize targetPosition, bool useAnchor, float scale, base::TimeDelta duration);
+ void requestStartPageScaleAnimationOnImplThread(gfx::Vector2d targetOffset, bool useAnchor, float scale, base::TimeDelta duration);
void finishAllRenderingOnImplThread(CompletionEvent*);
void initializeImplOnImplThread(CompletionEvent*, InputHandler*);
void setSurfaceReadyOnImplThread();