Chromium Code Reviews| Index: ui/android/overscroll_refresh.h |
| diff --git a/ui/android/overscroll_refresh.h b/ui/android/overscroll_refresh.h |
| index 7cb98e272ce711c843103d3b2ffa62d33b9296b5..07bf18db95f0e78efbad743b56cbd5f1b85ff3fe 100644 |
| --- a/ui/android/overscroll_refresh.h |
| +++ b/ui/android/overscroll_refresh.h |
| @@ -37,9 +37,10 @@ class UI_ANDROID_EXPORT OverscrollRefresh { |
| // Returns whether the refresh was activated. |
| void OnScrollEnd(const gfx::Vector2dF& velocity); |
| - // Scroll ack listener. The effect will only be activated if the initial |
| - // updates go unconsumed. |
| - void OnScrollUpdateAck(bool was_consumed); |
| + // Scroll ack listener. The effect will only be activated if the controller |
| + // decides it can navigate based on the consumed result and |
| + // scroll-boundary-behavior. |
|
majidvp
2017/07/20 13:26:55
Similar to the previous comment let's try to speci
sunyunjia
2017/07/21 13:52:23
Done.
|
| + virtual void OnOverscrolled(bool can_navigate); |
| // Returns true if the effect has consumed the |scroll_delta|. |
| bool WillHandleScrollUpdate(const gfx::Vector2dF& scroll_delta); |
| @@ -58,10 +59,10 @@ class UI_ANDROID_EXPORT OverscrollRefresh { |
| void Reset(); |
| // Returns true if the refresh effect is either being manipulated or animated. |
| - bool IsActive() const; |
| + virtual bool IsActive() const; |
| // Returns true if the effect is waiting for an unconsumed scroll to start. |
| - bool IsAwaitingScrollUpdateAck() const; |
| + virtual bool IsAwaitingScrollUpdateAck() const; |
| private: |
| void Release(bool allow_refresh); |