Moves WindowCycleList/Controller to ash/common

ash/common/wm_shell.cc

Index: ash/common/wm_shell.cc
diff --git a/ash/common/wm_shell.cc b/ash/common/wm_shell.cc
index 0bb9851c42ebbe41d4c0ed4e7cb60e16afa41a02..c1f101f28deb90f50849356403f2ce22857e1f63 100644
--- a/ash/common/wm_shell.cc
+++ b/ash/common/wm_shell.cc
@@ -18,6 +18,7 @@
 #include "ash/common/wm/maximize_mode/maximize_mode_controller.h"
 #include "ash/common/wm/mru_window_tracker.h"
 #include "ash/common/wm/overview/window_selector_controller.h"
+#include "ash/common/wm/window_cycle_controller.h"
 #include "ash/common/wm_window.h"
 #include "base/bind.h"
 #include "base/logging.h"
@@ -140,6 +141,14 @@ void WmShell::DeleteSystemTrayDelegate() {
   system_tray_delegate_.reset();
 }
 
+void WmShell::CreateWindowCycleController() {
+  window_cycle_controller_.reset(new WindowCycleController());

[James Cook, 2016/07/19 15:42:21]

It doesn't look like WindowCycleController does any interesting work in its
constructor and doesn't have any interesting member variables. Could it be
created in the WmShell constructor?

[sky, 2016/07/19 17:05:18]

Creation can happen at any time, but destruction depends upon
mru_window_tracker, so that has to be explicit. Updated.

+}
+
+void WmShell::DeleteWindowCycleController() {
+  window_cycle_controller_.reset();
+}
+
 void WmShell::DeleteWindowSelectorController() {
   window_selector_controller_.reset();
 }