[#5] Pass gfx structs by const ref (gfx::Vector2dF)

content/renderer/input/input_handler_proxy_unittest.cc

Index: content/renderer/input/input_handler_proxy_unittest.cc
diff --git a/content/renderer/input/input_handler_proxy_unittest.cc b/content/renderer/input/input_handler_proxy_unittest.cc
index e3616ba613fedf44f45a4c6ffc39c7f9336c99ba..8143db0ef0fc01cd7976e4e668fb22e693ce47c6 100644
--- a/content/renderer/input/input_handler_proxy_unittest.cc
+++ b/content/renderer/input/input_handler_proxy_unittest.cc
@@ -47,7 +47,8 @@ class MockInputHandler : public cc::InputHandler {
                ScrollStatus(gfx::Point viewport_point,
                             cc::InputHandler::ScrollInputType type));
   MOCK_METHOD2(ScrollBy,
-               bool(gfx::Point viewport_point, gfx::Vector2dF scroll_delta));
+               bool(gfx::Point viewport_point,
+                  const gfx::Vector2dF& scroll_delta));

[jdduke (slow), 2014/01/22 18:07:28]

Nit: Please align the second arg with the first (indent 2 more spaces).

   MOCK_METHOD2(ScrollVerticallyByPage,
                bool(gfx::Point viewport_point,
                     cc::ScrollDirection direction));
@@ -66,7 +67,8 @@ class MockInputHandler : public cc::InputHandler {
                                        float page_scale,
                                        base::TimeDelta duration) OVERRIDE {}
 
-  virtual void NotifyCurrentFlingVelocity(gfx::Vector2dF velocity) OVERRIDE {}
+  virtual void NotifyCurrentFlingVelocity(
+      const gfx::Vector2dF& velocity) OVERRIDE {}
   virtual void MouseMoveAt(gfx::Point mouse_position) OVERRIDE {}
 
   MOCK_METHOD1(HaveTouchEventHandlersAt,
@@ -77,7 +79,6 @@ class MockInputHandler : public cc::InputHandler {
       OVERRIDE {}
 
   virtual void OnRootLayerDelegatedScrollOffsetChanged() OVERRIDE {}
-

[danakj, 2014/01/22 17:39:43]

keep this line?

   DISALLOW_COPY_AND_ASSIGN(MockInputHandler);
 };