Separate SwipeRefreshHandler and ContentViewCore

ui/android/overscroll_refresh.h

 // Copyright 2014 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 UI_ANDROID_OVERSCROLL_REFRESH_H_
 #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
 //      offset and 2) lacks the overflow-y:hidden property.
 //   2) The page doesn't consume the initial scroll events.
 //   3) The initial scroll direction is upward.
 // The actuall pull response, animation and action are delegated to the
 // provided refresh handler.
 class UI_ANDROID_EXPORT OverscrollRefresh {
  public:
@@ skipping 41 matching lines @@
 
   bool scrolled_to_top_;
   bool overflow_y_hidden_;
 
   enum ScrollConsumptionState {
     DISABLED,
     AWAITING_SCROLL_UPDATE_ACK,
     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);
 };
 
 }  // namespace content
 
 #endif  // UI_ANDROID_OVERSCROLL_REFRESH_H_