Separate SwipeRefreshHandler and ContentViewCore

ui/android/overscroll_refresh.h

Index: ui/android/overscroll_refresh.h
diff --git a/ui/android/overscroll_refresh.h b/ui/android/overscroll_refresh.h
index 451805ed7c02c3d58b5a20ad3b7bdcab20887c5a..7aa79974e85d97f63720f6832d49d9c9571ca28b 100644
--- a/ui/android/overscroll_refresh.h
+++ b/ui/android/overscroll_refresh.h
@@ -6,33 +6,13 @@
 #define UI_ANDROID_OVERSCROLL_REFRESH_H_
 
 #include "base/macros.h"
+#include "ui/android/overscroll_refresh_handler.h"
 #include "ui/android/ui_android_export.h"
 #include "ui/gfx/geometry/size_f.h"
 #include "ui/gfx/geometry/vector2d_f.h"
 
 namespace ui {
 
-class UI_ANDROID_EXPORT OverscrollRefreshHandler {
- public:
-  // Signals the start of an overscrolling pull. Returns whether the handler
-  // will consume the overscroll gesture, in which case it will receive the
-  // remaining pull updates.
-  virtual bool PullStart() = 0;
-
-  // Signals a pull update, where |delta| is in device pixels.
-  virtual void PullUpdate(float delta) = 0;
-
-  // Signals the release of the pull, and whether the release is allowed to
-  // trigger the refresh action.
-  virtual void PullRelease(bool allow_refresh) = 0;
-
-  // Reset the active pull state.
-  virtual void PullReset() = 0;
-
- protected:
-  virtual ~OverscrollRefreshHandler() {}
-};
-
 // Simple pull-to-refresh styled effect. Listens to scroll events, conditionally
 // activating when:
 //   1) The scroll begins when the page's root layer 1) has no vertical scroll
@@ -94,7 +74,7 @@ class UI_ANDROID_EXPORT OverscrollRefresh {
     ENABLED,
   } scroll_consumption_state_;
 
-  OverscrollRefreshHandler* const handler_;
+  OverscrollRefreshHandler* handler_;

[boliu, 2016/12/06 00:58:16]

can stay const I think?

[rlanday, 2016/12/06 03:25:26]

Yep

 
   DISALLOW_COPY_AND_ASSIGN(OverscrollRefresh);
 };