message-center: Fix fling scroll in the list.

message-center: Fix fling scroll in the list.

BUG=263627
R=mukai@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=213335

[Jun Mukai, 2013-07-24 00:22:22]

lgtm

So it's in message_center... sigh.
Thank you very much! (and sorry for blaming you)

[Jun Mukai, 2013-07-24 00:52:08]

https://codereview.chromium.org/19471016/diff/1/ui/message_center/views/messa...
File ui/message_center/views/message_view.cc (right):

https://codereview.chromium.org/19471016/diff/1/ui/message_center/views/messa...
ui/message_center/views/message_view.cc:434: event->type() !=
ui::ET_SCROLL_FLING_START)
by the way, should it also check ET_SCROLL_FLING_CANCEL in case the user wants
to cancel the fling scroll?

[sadrul, 2013-07-24 04:45:17]

Committed patchset #2 manually as r213335 (presubmit successful).

[sadrul, 2013-07-24 04:46:42]

https://codereview.chromium.org/19471016/diff/1/ui/message_center/views/messa...
File ui/message_center/views/message_view.cc (right):

https://codereview.chromium.org/19471016/diff/1/ui/message_center/views/messa...
ui/message_center/views/message_view.cc:434: event->type() !=
ui::ET_SCROLL_FLING_START)
On 2013/07/24 00:52:08, Jun Mukai wrote:
> by the way, should it also check ET_SCROLL_FLING_CANCEL in case the user wants
> to cancel the fling scroll?

Done.

(The gesture-recognizer never generates a FLING_CANCEL event. So confusingly
enough, we don't actually need to check for this. But it's better to check for
that for readability if nothing else.)