[ash-md] Adds support for gesture to move selection in overview mode

ash/wm/gestures/overview_gesture_handler_unittest.cc

 // Copyright 2013 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 "ash/wm/gestures/overview_gesture_handler.h"
+
 #include "ash/common/wm/overview/window_selector_controller.h"
 #include "ash/common/wm_shell.h"
 #include "ash/root_window_controller.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
+#include "ash/wm/window_util.h"
 #include "ui/aura/test/test_window_delegate.h"
 #include "ui/aura/test/test_windows.h"
 #include "ui/aura/window.h"
 #include "ui/aura/window_event_dispatcher.h"
 #include "ui/events/test/event_generator.h"
 #include "ui/views/widget/widget.h"
 
 namespace ash {
 
 class OverviewGestureHandlerTest : public test::AshTestBase {
  public:
   OverviewGestureHandlerTest() {}
   ~OverviewGestureHandlerTest() override {}
 
   aura::Window* CreateWindow(const gfx::Rect& bounds) {
     return CreateTestWindowInShellWithDelegate(&delegate_, -1, bounds);
   }
 
   bool IsSelecting() {
     return WmShell::Get()->window_selector_controller()->IsSelecting();
   }
 
+  float vertical_threshold_pixels() const {
+    return OverviewGestureHandler::vertical_threshold_pixels_;
+  }
+
+  float horizontal_threshold_pixels() const {
+    return OverviewGestureHandler::horizontal_threshold_pixels_;
+  }
+
  private:
   aura::test::TestWindowDelegate delegate_;
 
   DISALLOW_COPY_AND_ASSIGN(OverviewGestureHandlerTest);
 };
 
 // Tests a swipe up with three fingers to enter and a swipe down to exit
 // overview.
 TEST_F(OverviewGestureHandlerTest, VerticalSwipes) {
   gfx::Rect bounds(0, 0, 400, 400);
   aura::Window* root_window = Shell::GetPrimaryRootWindow();
   std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
   std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
   ui::test::EventGenerator generator(root_window, root_window);
+  const float long_swipe = 2 * vertical_threshold_pixels();
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           0, -500, 100, 3);
+                           0, -long_swipe, 100, 3);
   EXPECT_TRUE(IsSelecting());
 
   // Swiping up again does nothing.
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           0, -500, 100, 3);
+                           0, -long_swipe, 100, 3);
   EXPECT_TRUE(IsSelecting());
 
   // Swiping down exits.
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           0, 500, 100, 3);
+                           0, long_swipe, 100, 3);
   EXPECT_FALSE(IsSelecting());
 
   // Swiping down again does nothing.
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           0, 500, 100, 3);
+                           0, long_swipe, 100, 3);
   EXPECT_FALSE(IsSelecting());
 }
 
+// Tests a swipe up with three fingers to enter overview and a swipe right or

[tdanderson, 2017/02/08 23:14:19]

The OverviewGestureHandlerTest.VerticalSwipes test already covers the case of
'three-finger scroll up/down will enter/exit overview mode'. In this test,
consider engaging overview mode by some other means to make it clear that the
horizontal n-finger scrolling while in overview mode does not have the vertical
3-finger scroll as a prerequisite.

[varkha, 2017/02/09 21:54:42]

Done.

+// left to move selection.
+TEST_F(OverviewGestureHandlerTest, VerticalThenHorizontalSwipe) {
+  gfx::Rect bounds(0, 0, 400, 400);
+  aura::Window* root_window = Shell::GetPrimaryRootWindow();
+  std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
+  std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
+  std::unique_ptr<aura::Window> window3(CreateWindow(bounds));
+  std::unique_ptr<aura::Window> window4(CreateWindow(bounds));
+  std::unique_ptr<aura::Window> window5(CreateWindow(bounds));
+  ui::test::EventGenerator generator(root_window, root_window);
+  const float vertical_swipe = 2 * vertical_threshold_pixels();
+  const float horizontal_swipe = horizontal_threshold_pixels();
+  generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
+                           0, -vertical_swipe, 100, 3);
+  EXPECT_TRUE(IsSelecting());
+
+  // Long swipe right moves selection to the fourth window.
+  generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
+                           horizontal_swipe * 4, 0, 100, 3);
+  EXPECT_TRUE(IsSelecting());
+
+  // Short swipe left (3 fingers) moves selection to the third window.
+  generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
+                           -horizontal_swipe, 0, 100, 3);
+  EXPECT_TRUE(IsSelecting());
+
+  // Short swipe left (2 fingers) moves selection to the second window.
+  generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
+                           -horizontal_swipe, 0, 100, 2);
+  EXPECT_TRUE(IsSelecting());
+
+  // Swiping down exits and selects the currently-highlighted window.
+  generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
+                           0, vertical_swipe, 100, 3);
+  EXPECT_FALSE(IsSelecting());
+
+  // Second MRU window is selected (i.e. |window4|).
+  EXPECT_EQ(window4.get(), wm::GetActiveWindow());
+}
+
 // Tests that a mostly horizontal swipe does not trigger overview.

[tdanderson, 2017/02/08 23:14:19]

nit: while you're here it would be good to reword 'swipe' as you have done
elsewhere so that all documentation is consistent.

[varkha, 2017/02/09 21:54:42]

Done.

 TEST_F(OverviewGestureHandlerTest, HorizontalSwipes) {
   gfx::Rect bounds(0, 0, 400, 400);
   aura::Window* root_window = Shell::GetPrimaryRootWindow();
   std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
   std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
   ui::test::EventGenerator generator(root_window, root_window);
+  const float long_swipe = 2 * vertical_threshold_pixels();
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           600, -500, 100, 3);
+                           long_swipe + 100, -long_swipe, 100, 3);
   EXPECT_FALSE(IsSelecting());
 
   generator.ScrollSequence(gfx::Point(), base::TimeDelta::FromMilliseconds(5),
-                           -600, -500, 100, 3);
+                           -long_swipe - 100, -long_swipe, 100, 3);
   EXPECT_FALSE(IsSelecting());
 }
 
 // Tests a swipe up with three fingers without releasing followed by a swipe
 // down by a lesser amount which should still be enough to exit.
 TEST_F(OverviewGestureHandlerTest, SwipeUpDownWithoutReleasing) {
   gfx::Rect bounds(0, 0, 400, 400);
   aura::Window* root_window = Shell::GetPrimaryRootWindow();
   std::unique_ptr<aura::Window> window1(CreateWindow(bounds));
   std::unique_ptr<aura::Window> window2(CreateWindow(bounds));
@@ skipping 24 matching lines @@
     generator.Dispatch(&move);
   }
 
   EXPECT_FALSE(IsSelecting());
   ui::ScrollEvent fling_start(ui::ET_SCROLL_FLING_START, start, timestamp, 0, 0,
                               10, 0, 10, num_fingers);
   generator.Dispatch(&fling_start);
 }
 
 }  // namespace ash