exo: Drop window on touch release

components/exo/shell_surface.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/shell_surface.h"
 
 #include <algorithm>
 
 #include "ash/frame/custom_frame_view_ash.h"
 #include "ash/public/cpp/shelf_types.h"
@@ skipping 1100 matching lines @@
     case ui::ET_MOUSE_EXITED:
     case ui::ET_MOUSEWHEEL:
     case ui::ET_MOUSE_CAPTURE_CHANGED:
       break;
     default:
       NOTREACHED();
       break;
   }
 }
 
+void ShellSurface::OnGestureEvent(ui::GestureEvent* event) {
+  if (!resizer_) {
+    views::View::OnGestureEvent(event);
+    return;
+  }
+
+  if (event->handled())
+    return;
+
+  switch (event->type()) {
+    case ui::ET_GESTURE_END: {
+      ScopedConfigure scoped_configure(this, false);
+      EndDrag(false /* revert */);
+      break;
+    }
+

[reveman, 2017/06/30 21:48:16]

nit: remove this blank line

[Dominik Laskowski, 2017/06/30 22:27:57]

Done.

+    default:
+      DCHECK_GE(event->type(), ui::ET_GESTURE_TYPE_START);

[reveman, 2017/06/30 21:48:16]

either add all enum values above even if there's a lot of them or switch to a
simple "if (event->type() == ui::ET_GESTURE_END)", the switch statement doesn't
provide any value otherwise imo

[Dominik Laskowski, 2017/06/30 22:27:57]

Done.

+      DCHECK_LE(event->type(), ui::ET_GESTURE_TYPE_END);
+      break;
+  }
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // ui::AcceleratorTarget overrides:
 
 bool ShellSurface::AcceleratorPressed(const ui::Accelerator& accelerator) {
   for (const auto& entry : kCloseWindowAccelerators) {
     if (ui::Accelerator(entry.keycode, entry.modifiers) == accelerator) {
       if (!close_callback_.is_null())
         close_callback_.Run();
       return true;
     }
@@ skipping 550 matching lines @@
 gfx::Point ShellSurface::GetMouseLocation() const {
   aura::Window* const root_window = widget_->GetNativeWindow()->GetRootWindow();
   gfx::Point location =
       root_window->GetHost()->dispatcher()->GetLastMouseLocationInRoot();
   aura::Window::ConvertPointToTarget(
       root_window, widget_->GetNativeWindow()->parent(), &location);
   return location;
 }
 
 }  // namespace exo