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

ash/common/wm/overview/window_selector_controller.cc

Index: ash/common/wm/overview/window_selector_controller.cc
diff --git a/ash/common/wm/overview/window_selector_controller.cc b/ash/common/wm/overview/window_selector_controller.cc
index 28931f21ddd77bb29e7fa4083313a068197cdc14..2c8b4efa2cf05370fa66a9ea48442e2d6f419fd5 100644
--- a/ash/common/wm/overview/window_selector_controller.cc
+++ b/ash/common/wm/overview/window_selector_controller.cc
@@ -75,6 +75,16 @@ bool WindowSelectorController::IsSelecting() const {
   return window_selector_.get() != NULL;
 }
 
+void WindowSelectorController::IncrementSelection(int increment) {
+  DCHECK(IsSelecting());
+  window_selector_->IncrementSelection(increment);
+}
+
+bool WindowSelectorController::AcceptSelection() {
+  DCHECK(IsSelecting());

[tdanderson, 2017/02/06 20:13:57]

Is this necessary given how you've documented AcceptSelection(), and given line
418 in window_selector.cc?

[varkha, 2017/02/08 20:41:13]

WindowSelectorController::IsSelecting is true when we are in overview mode, not
necessarily when a selection is made (selector widget present). So yes, it
confirms that we are in overview mode and can dereference window_selector_.
Maybe somewhat redundant but the crash won't happen immediately, it will happen
lower in stack and might be not obvious to grok, hence the DCHECK.

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

Acknowledged.

+  return window_selector_->AcceptSelection();
+}
+
 bool WindowSelectorController::IsRestoringMinimizedWindows() const {
   return window_selector_.get() != NULL &&
          window_selector_->restoring_minimized_windows();