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..1e06bc1f2fd8f474ee5660506968973c5aa18f1b 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 |can_navigate| |
| + // is true which happens when the scroll update is not consumed and the |
| + // scroll_boundary_behavior on y axis is 'auto'. |
|
boliu
2017/08/23 19:17:03
comment on virtual methods that they are virtual f
sunyunjia
2017/08/23 23:47:06
Done.
|
| + virtual void OnOverscrolled(); |
| // Returns true if the effect has consumed the |scroll_delta|. |
| bool WillHandleScrollUpdate(const gfx::Vector2dF& scroll_delta); |
| @@ -55,13 +56,13 @@ class UI_ANDROID_EXPORT OverscrollRefresh { |
| // Reset the effect to its inactive state, immediately detaching and |
| // disabling any active effects. |
| - void Reset(); |
| + virtual 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); |