Remove TOGGLE_MAXIMIZED_* from Ash reserved actions list.

chrome/browser/ui/ash/chrome_shell_delegate.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/ash/chrome_shell_delegate.h"
 
 #include "ash/launcher/launcher_types.h"
 #include "ash/system/tray/system_tray_delegate.h"
 #include "ash/wm/window_util.h"
 #include "base/command_line.h"
@@ skipping 164 matching lines @@
   if (!was_active)
     browser->window()->Show();
 }
 
 void ChromeShellDelegate::NewWindow(bool is_incognito) {
   Profile* profile = ProfileManager::GetDefaultProfileOrOffTheRecord();
   chrome::NewEmptyWindow(
       is_incognito ? profile->GetOffTheRecordProfile() : profile);
 }
 
+void ChromeShellDelegate::ToggleMaximized() {
+  aura::Window* window = ash::wm::GetActiveWindow();
+  if (!window)
+    return;
+  // Get out of fullscreen when in fullscreen mode.
+  if (ash::wm::IsWindowFullscreen(window)) {
+    chrome::ToggleFullscreenMode(GetTargetBrowser());
+    return;
+  }
+  if (ash::wm::IsWindowMaximized(window))

[sky, 2012/10/09 15:50:53]

Promote this to a common place so TestShellDelegate and this can share it.

[mazda, 2012/10/09 20:50:33]

I added ToggleMaximizedWindow to ash/wm/window_util.h and moved this part to its
implementation. Does this sound good?

+    ash::wm::RestoreWindow(window);
+  else if (ash::wm::CanMaximizeWindow(window))
+    ash::wm::MaximizeWindow(window);
+}
+
 void ChromeShellDelegate::OpenFileManager(bool as_dialog) {
 #if defined(OS_CHROMEOS)
   if (as_dialog) {
     Browser* browser =
         browser::FindBrowserWithWindow(ash::wm::GetActiveWindow());
     // Open the select file dialog only if there is an active browser where the
     // selected file is displayed. Otherwise open a file manager in a tab.
     if (browser) {
       browser->OpenFile();
       return;
@@ skipping 255 matching lines @@
       ash::Shell::GetInstance()->ShowLauncher();
       break;
     default:
       NOTREACHED() << "Unexpected notification " << type;
   }
 #else
   // MSVC++ warns about switch statements without any cases.
   NOTREACHED() << "Unexpected notification " << type;
 #endif
 }