Pass gfx structs by const ref (gfx::Vector2dF)

Avoid unneccessary copy of structures gfx::Vector2dF
by passing them by const ref rather than value.

Any struct of size > 4 bytes should be passed by const ref.
Passing by ref for these structs is faster than passing
by value, especially when invoking function has multiple parameters.


Pass gfx structs by const ref (gfx::Vector2dF)

BUG=159273

Review URL: https://codereview.chromium.org/130443005

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@246563 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 2 deletions

diff --git a/cc/test/animation_test_common.h b/cc/test/animation_test_common.h
index 1205bf3..e23be0b 100644
--- a/cc/test/animation_test_common.h
+++ b/cc/test/animation_test_common.h
@@ -75,7 +75,8 @@ class FakeLayerAnimationValueObserver : public LayerAnimationValueObserver {
   virtual void OnFilterAnimated(const FilterOperations& filters) OVERRIDE;
   virtual void OnOpacityAnimated(float opacity) OVERRIDE;
   virtual void OnTransformAnimated(const gfx::Transform& transform) OVERRIDE;
-  virtual void OnScrollOffsetAnimated(gfx::Vector2dF scroll_offset) OVERRIDE;
+  virtual void OnScrollOffsetAnimated(
+      const gfx::Vector2dF& scroll_offset) OVERRIDE;
   virtual void OnAnimationWaitingForDeletion() OVERRIDE;
   virtual bool IsActive() const OVERRIDE;
 
@@ -106,7 +107,7 @@ class FakeLayerAnimationValueProvider : public LayerAnimationValueProvider {
  public:
   virtual gfx::Vector2dF ScrollOffsetForAnimation() const OVERRIDE;
 
-  void set_scroll_offset(gfx::Vector2dF scroll_offset) {
+  void set_scroll_offset(const gfx::Vector2dF& scroll_offset) {
     scroll_offset_ = scroll_offset;
   }