Use DidOverscrollParams in the browser

content/port/browser/render_widget_host_view_port.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_
 #define CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_
 
 #include "base/basictypes.h"
 #include "base/callback.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 26 matching lines @@
 
 namespace blink {
 struct WebScreenInfo;
 }
 
 namespace content {
 class RenderWidgetHostViewFrameSubscriber;
 class SyntheticGesture;
 class SyntheticGestureTarget;
 class WebCursor;
+struct DidOverscrollParams;
 struct WebPluginGeometry;
 struct NativeWebKeyboardEvent;
 
 // This is the larger RenderWidgetHostView interface exposed only
 // within content/ and to embedders looking to port to new platforms.
 // RenderWidgetHostView class hierarchy described in render_widget_host_view.h.
 class CONTENT_EXPORT RenderWidgetHostViewPort : public RenderWidgetHostView,
                                                 public IPC::Listener {
  public:
   virtual ~RenderWidgetHostViewPort() {}
@@ skipping 244 matching lines @@
   // Called by the host when the input flush has completed.
   virtual void OnDidFlushInput() = 0;
 
   // Create a platform specific SyntheticGestureTarget implementation that will
   // be used to inject synthetic input events.
   virtual scoped_ptr<SyntheticGestureTarget> CreateSyntheticGestureTarget() = 0;
 
   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;
 
   virtual void SetPopupType(blink::WebPopupType popup_type) = 0;
   virtual blink::WebPopupType GetPopupType() = 0;
 
   // Get the BrowserAccessibilityManager if it exists, may return NULL.
   virtual BrowserAccessibilityManager*
       GetBrowserAccessibilityManager() const = 0;
   // Create a BrowserAccessibilityManager for this view if it's possible to
@@ skipping 30 matching lines @@
 
   // Returns an HWND that's given as the parent window for windowless Flash to
   // workaround crbug.com/301548.
   virtual gfx::NativeViewId GetParentForWindowlessPlugin() const = 0;
 #endif
 };
 
 }  // namespace content
 
 #endif  // CONTENT_PORT_BROWSER_RENDER_WIDGET_HOST_VIEW_PORT_H_