Use DidOverscrollParams in the browser

content/port/browser/render_widget_host_view_port.h

Index: content/port/browser/render_widget_host_view_port.h
diff --git a/content/port/browser/render_widget_host_view_port.h b/content/port/browser/render_widget_host_view_port.h
index 3b860c7e6a56382862eff02db389087e4a49c8db..2eb90cf04eeae4aef7e79d5e038a052696fc5e6a 100644
--- a/content/port/browser/render_widget_host_view_port.h
+++ b/content/port/browser/render_widget_host_view_port.h
@@ -44,6 +44,7 @@ class RenderWidgetHostViewFrameSubscriber;
 class SyntheticGesture;
 class SyntheticGestureTarget;
 class WebCursor;
+struct DidOverscrollParams;
 struct WebPluginGeometry;
 struct NativeWebKeyboardEvent;
 
@@ -308,8 +309,7 @@ class CONTENT_EXPORT RenderWidgetHostViewPort : public RenderWidgetHostView,
   virtual void GestureEventAck(const blink::WebGestureEvent& event,
                                InputEventAckState ack_result) = 0;
 
-  virtual void OnOverscrolled(gfx::Vector2dF accumulated_overscroll,
-                              gfx::Vector2dF current_fling_velocity) = 0;
+  virtual void DidOverscroll(const DidOverscrollParams& params) = 0;

[jam, 2014/04/25 05:09:07]

any struct used in content/port must be in content/port or content/public
(otherwise an embedder using this would have to reach into the internal
implementation of content). move this struct to content/port/common

[jdduke (slow), 2014/04/25 14:44:33]

Hmm, I wonder if I should remove this method entirely.  It's only used on
Android, and will likely remain that way for the foreseeable future, so I'll
just handle the message in RWHVA without cluttering the view.  I'll amend the
patch.

 
   virtual void DidStopFlinging() = 0;