Move MaximizeModeWindowManager to the controller

ash/wm/maximize_mode/maximize_mode_controller_unittest.cc

Index: ash/wm/maximize_mode/maximize_mode_controller_unittest.cc
diff --git a/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc b/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc
index e9d925d729a7d322a761b1709079e1d4b82c07ef..5ee5de1f5e602b6421eee9ccfad5d84036694979 100644
--- a/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc
+++ b/ash/wm/maximize_mode/maximize_mode_controller_unittest.cc
@@ -113,7 +113,7 @@ class MaximizeModeControllerTest : public test::AshTestBase {
     test::AshTestBase::TearDown();
   }
 
-  MaximizeModeController* maximize_mode_controller() {
+  MaximizeModeController* maximize_mode_controller() const {

[flackr, 2014/05/29 16:10:06]

How about const and non-const versions which return a const or non const
pointer?

See for example Window::GetRootWindow. Although the implementation is short
enough you can do it twice instead of a const cast.

[jonross, 2014/05/29 17:40:00]

Removed the original const on 1125 to clear the need here. That way there is no
const/non-const mix up with the pointer.

     return Shell::GetInstance()->maximize_mode_controller();
   }
 
@@ -123,7 +123,7 @@ class MaximizeModeControllerTest : public test::AshTestBase {
   }
 
   bool IsMaximizeModeStarted() const {
-    return Shell::GetInstance()->IsMaximizeModeWindowManagerEnabled();
+    return maximize_mode_controller()->IsMaximizeModeWindowManagerEnabled();
   }
 
   // Overrides the internal input device list for the current event targeters