exo: Replace fling cancel with generic zero distance scroll

components/exo/pointer.cc

 // Copyright 2015 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 "components/exo/pointer.h"
 
 #include "ash/public/cpp/shell_window_ids.h"
 #include "cc/output/copy_output_request.h"
 #include "cc/output/copy_output_result.h"
 #include "components/exo/pointer_delegate.h"
@@ skipping 194 matching lines @@
           static_cast<ui::MouseWheelEvent*>(event)->offset(), true);
       delegate_->OnPointerFrame();
       break;
     }
     case ui::ET_SCROLL_FLING_START: {
       delegate_->OnPointerScrollStop(event->time_stamp());
       delegate_->OnPointerFrame();
       break;
     }
     case ui::ET_SCROLL_FLING_CANCEL: {
-      delegate_->OnPointerScrollCancel(event->time_stamp());
-      delegate_->OnPointerFrame();
+      if (last_event_type_ == ui::ET_SCROLL_FLING_START) {

[reveman, 2017/04/27 13:28:17]

Can you add a comment here that explains why we do this?

[denniskempin (chromium), 2017/05/03 20:55:28]

Done.

+        delegate_->OnPointerScroll(event->time_stamp(), gfx::Vector2dF(),

[reveman, 2017/04/27 13:28:17]

I think this logic deserves a comment similar to what we had in server.cc

[denniskempin (chromium), 2017/05/03 20:55:28]

Done.

+                                   false);
+        delegate_->OnPointerFrame();
+        delegate_->OnPointerScrollStop(event->time_stamp());
+        delegate_->OnPointerFrame();
+      }
       break;
     }
     case ui::ET_MOUSE_MOVED:
     case ui::ET_MOUSE_DRAGGED:
     case ui::ET_MOUSE_ENTERED:
     case ui::ET_MOUSE_EXITED:
     case ui::ET_MOUSE_CAPTURE_CHANGED:
       break;
     default:
       NOTREACHED();
       break;
   }
 
+  last_event_type_ = event->type();
   UpdateCursorScale();
 }
 
 void Pointer::OnScrollEvent(ui::ScrollEvent* event) {
   OnMouseEvent(event);
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // WMHelper::CursorObserver overrides:
 
@@ skipping 142 matching lines @@
   if (!root_window)
     return;
 
   aura::client::CursorClient* cursor_client =
       aura::client::GetCursorClient(root_window);
   if (cursor_client)
     cursor_client->SetCursor(cursor_);
 }
 
 }  // namespace exo