[Cleanup] Rename WindowSettings to WindowState

ash/test/test_shell_delegate.cc

Index: ash/test/test_shell_delegate.cc
diff --git a/ash/test/test_shell_delegate.cc b/ash/test/test_shell_delegate.cc
index 4bd60838d724cdf8b635ce39f6303f63fab4adea..85851eb19b2745476f28ccd05c3a3da54897577e 100644
--- a/ash/test/test_shell_delegate.cc
+++ b/ash/test/test_shell_delegate.cc
@@ -15,6 +15,7 @@
 #include "ash/test/test_launcher_delegate.h"
 #include "ash/test/test_session_state_delegate.h"
 #include "ash/test/test_system_tray_delegate.h"
+#include "ash/wm/window_state.h"
 #include "ash/wm/window_util.h"
 #include "base/logging.h"
 #include "content/public/test/test_browser_context.h"
@@ -67,9 +68,9 @@ void TestShellDelegate::NewWindow(bool incognito) {
 }
 
 void TestShellDelegate::ToggleMaximized() {
-  aura::Window* window = ash::wm::GetActiveWindow();
-  if (window)
-    ash::wm::ToggleMaximizedWindow(window);
+  wm::WindowState* window_state = wm::GetActiveWindowState();

[James Cook, 2013/09/18 20:44:17]

nit: Are you sure you want to provide GetActiveWindowState(), which does two
actions in one call?  I don't think it would be that bad to keep
GetActiveWindow() and call window->GetWindowState()->ToggleMaximized().

[oshima, 2013/09/19 01:52:01]

I added because most cases you get active window, you ends up getting
window_state.

+  if (window_state)
+    window_state->ToggleMaximized();
 }
 
 void TestShellDelegate::ToggleFullscreen() {