message-center: Fix fling scroll in the list.

ui/message_center/views/message_view.cc

 // Copyright (c) 2012 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.
 
 #include "ui/message_center/views/message_view.h"
 
 #include "grit/ui_resources.h"
 #include "grit/ui_strings.h"
 #include "ui/base/accessibility/accessible_view_state.h"
 #include "ui/base/l10n/l10n_util.h"
@@ skipping 412 matching lines @@
     message_center_->ClickOnNotification(notification_id_);
     event->SetHandled();
     return;
   }
 
   SlideOutView::OnGestureEvent(event);
   // Do not return here by checking handled(). SlideOutView calls SetHandled()
   // even though the scroll gesture doesn't make no (or little) effects on the
   // slide-out behavior. See http://crbug.com/172991
 
-  if (!event->IsScrollGestureEvent())
+  if (!event->IsScrollGestureEvent() &&
+      event->type() != ui::ET_SCROLL_FLING_START)

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

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:46:43]

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.)

     return;
 
   if (scroller_)
     scroller_->OnGestureEvent(event);
   event->SetHandled();
 }
 
 void MessageView::OnPaintFocusBorder(gfx::Canvas* canvas) {
   if (HasFocus()) {
     canvas->DrawRect(gfx::Rect(1, 0, width() - 2, height() - 2),
                      message_center::kFocusBorderColor);
   }
 }
 
 void MessageView::ButtonPressed(views::Button* sender,
                                 const ui::Event& event) {
   if (sender == close_button()) {
     message_center_->RemoveNotification(notification_id_, true);  // By user.
   } else if (sender == expand_button()) {
     is_expanded_ = true;
     message_center_->ExpandNotification(notification_id_);
   }
 }
 
 void MessageView::OnSlideOut() {
   message_center_->RemoveNotification(notification_id_, true);  // By user.
 }
 
 }  // namespace message_center