Combined window positioning and show state determiniation in the WindowSizer

chrome/browser/ui/window_sizer/window_sizer_ash_unittest.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/window_sizer/window_sizer.h"
-
-#include <vector>
+#include "chrome/browser/ui/window_sizer/window_sizer_common_unittest.h"
 
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/test/test_shell_delegate.h"
 #include "ash/wm/window_resizer.h"
 #include "base/compiler_specific.h"
-#include "base/logging.h"
 #include "chrome/browser/ui/browser.h"
-#include "chrome/browser/ui/window_sizer/window_sizer_common_unittest.h"
+#include "chrome/common/chrome_switches.h"
 #include "chrome/test/base/testing_profile.h"
-#include "chrome/test/base/test_browser_window.h"
-#include "content/public/browser/browser_thread.h"
 #include "content/public/test/render_view_test.h"
-#include "content/public/test/test_browser_thread.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "third_party/WebKit/Source/WebKit/chromium/public/WebKit.h"
+#include "ui/aura/client/aura_constants.h"
 #include "ui/aura/env.h"
 #include "ui/aura/root_window.h"
 #include "ui/aura/test/test_windows.h"
 
+typedef ash::test::AshTestBase WindowSizerTest;
+
 namespace {
 
-typedef ash::test::AshTestBase WindowSizerTest;
+// A browser window proxy which is able to associate an aura native window with
+// it.
+class TestBrowserWindowAura : public TestBrowserWindow {
+ public:
+  // |native_window| will still be owned by the caller after the constructor
+  // was called.
+  explicit TestBrowserWindowAura(aura::Window* native_window);
+  virtual ~TestBrowserWindowAura();
+
+  virtual gfx::NativeWindow GetNativeWindow() OVERRIDE;
+
+  virtual gfx::Rect GetBounds() const OVERRIDE;
+
+ private:
+  // The |native_window_| is still owned by the creator of this class.
+  aura::Window* native_window_;
+
+  DISALLOW_COPY_AND_ASSIGN(TestBrowserWindowAura);
+};
+
+TestBrowserWindowAura::TestBrowserWindowAura(aura::Window *native_window)
+    : native_window_(native_window) {
+}
+
+TestBrowserWindowAura::~TestBrowserWindowAura() {}
+
+gfx::NativeWindow TestBrowserWindowAura::GetNativeWindow() {
+    return native_window_;
+}
+
+gfx::Rect TestBrowserWindowAura::GetBounds() const {
+  return native_window_->bounds();
+}
+
+int AlignToGridRoundDown(int location, int grid_size) {
+  if (grid_size <= 1 || location % grid_size == 0)
+    return location;
+  return location / grid_size * grid_size;
+}
 
 // A special test class for use with browser creation - it will create a
 // browser thread and deletes it after all other things have been destroyed.
 class WindowSizerTestWithBrowser : public WindowSizerTest {
  public:
   WindowSizerTestWithBrowser();
   virtual ~WindowSizerTestWithBrowser();
 
  private:
   // Note: It is important to delete the thread after the browser instances got
   // deleted. For this we transfer the thread here.
   scoped_ptr<content::TestBrowserThread> ui_thread_;
 
   DISALLOW_COPY_AND_ASSIGN(WindowSizerTestWithBrowser);
 };
 
+// The class function definitions from window_sizer_common_unittest.h
 WindowSizerTestWithBrowser::WindowSizerTestWithBrowser() {
   // Set up a UI message thread.
   MessageLoopForUI* ui_loop = message_loop();
   ui_thread_.reset(
       new content::TestBrowserThread(content::BrowserThread::UI, ui_loop));
 }
 
 WindowSizerTestWithBrowser::~WindowSizerTestWithBrowser() {
 }
 
-// A browser window proxy which is able to associate an aura native window with
-// it.
-class TestBrowserWindowAura : public TestBrowserWindow {
- public:
-  explicit TestBrowserWindowAura(aura::Window* native_window);
-  virtual ~TestBrowserWindowAura();
-
-  virtual gfx::NativeWindow GetNativeWindow() OVERRIDE {
-    return native_window_;
-  }
-
-  virtual gfx::Rect GetBounds() const {
-    return native_window_->bounds();
-  }
-
- private:
-  gfx::NativeWindow native_window_;
-
-  DISALLOW_COPY_AND_ASSIGN(TestBrowserWindowAura);
-};
-
-} // namespace
-
-TestBrowserWindowAura::TestBrowserWindowAura(aura::Window *native_window)
-    : native_window_(native_window) {
 }
-
-TestBrowserWindowAura::~TestBrowserWindowAura() {}
-
-int AlignToGridRoundDown(int location, int grid_size) {
-  if (grid_size <= 1 || location % grid_size == 0)
-    return location;
-  return location / grid_size * grid_size;
-}
-
 // Test that the window is sized appropriately for the first run experience
 // where the default window bounds calculation is invoked.
 TEST_F(WindowSizerTest, DefaultSizeCase) {
   int grid = WindowSizer::kDesktopBorderSize;
   { // 4:3 monitor case, 1024x768, no taskbar
     gfx::Rect window_bounds;
     GetWindowBounds(tentwentyfour, tentwentyfour, gfx::Rect(), gfx::Rect(),
                     gfx::Rect(), DEFAULT, &window_bounds, NULL, gfx::Rect());
     EXPECT_EQ(gfx::Rect(WindowSizer::kDesktopBorderSize,
                         WindowSizer::kDesktopBorderSize,
@@ skipping 654 matching lines @@
   }
 
   { // Check that a window which hangs out of the screen get moved back in.
     gfx::Rect window_bounds;
     GetWindowBounds(tentwentyfour, tentwentyfour, gfx::Rect(), gfx::Rect(),
                     gfx::Rect(), DEFAULT, &window_bounds, NULL,
                     gfx::Rect(1020, 700, 100, 100));
     EXPECT_EQ("924,668 100x100", window_bounds.ToString());
   }
 }
+
+// Test that the show state is properly returned for non default cases.
+TEST_F(WindowSizerTestWithBrowser, TestShowState) {
+  // Creating a browser & window to play with.
+  scoped_ptr<aura::Window> window(
+      aura::test::CreateTestWindowWithId(0, NULL));
+  window->SetBounds(gfx::Rect(16, 32, 640, 320));
+
+  scoped_ptr<TestingProfile> profile(new TestingProfile());
+
+  scoped_ptr<BrowserWindow> browser_window(
+      new TestBrowserWindowAura(window.get()));
+  Browser::CreateParams window_params(Browser::TYPE_TABBED, profile.get());
+  window_params.window = browser_window.get();
+  scoped_ptr<Browser> browser(new Browser(window_params));
+
+  // Create also a popup browser since that behaves different.
+  scoped_ptr<aura::Window> popup(
+      aura::test::CreateTestWindowWithId(1, NULL));
+  popup->SetBounds(gfx::Rect(16, 32, 128, 256));
+
+  scoped_ptr<BrowserWindow> browser_popup(
+      new TestBrowserWindowAura(popup.get()));
+  Browser::CreateParams popup_params(Browser::TYPE_POPUP, profile.get());
+  popup_params.window = browser_window.get();
+  scoped_ptr<Browser> popup_browser(new Browser(popup_params));
+
+  // Tabbed windows should retrieve the saved window state - since there is a
+  // top window.
+  EXPECT_EQ(ui::SHOW_STATE_MAXIMIZED,
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               browser.get()));
+  EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
+            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               browser.get()));
+  // Non tabbed windows should always follow the window saved visibility state.
+  EXPECT_EQ(ui::SHOW_STATE_MAXIMIZED,
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               popup_browser.get()));
+  // The non tabbed window will take the status of the last active of its kind.
+  EXPECT_EQ(ui::SHOW_STATE_NORMAL,
+            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               popup_browser.get()));
+
+  // Now create a top level window and check again for both. Only the tabbed
+  // window should follow the top level window's state.
+  // Creating a browser & window to play with.
+  scoped_ptr<aura::Window> window2(
+      aura::test::CreateTestWindowWithId(0, NULL));
+  window->SetBounds(gfx::Rect(16, 32, 640, 320));
+
+  scoped_ptr<BrowserWindow> browser_window2(
+      new TestBrowserWindowAura(window2.get()));
+  Browser::CreateParams window2_params(Browser::TYPE_TABBED, profile.get());
+  window2_params.window = browser_window2.get();
+  scoped_ptr<Browser> browser2(new Browser(window2_params));
+
+  // A tabbed window should now take the top level window state.
+  EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_DEFAULT,
+                               BOTH,
+                               browser2.get()));
+  // Non tabbed windows should always follow the window saved visibility state.
+  EXPECT_EQ(ui::SHOW_STATE_MAXIMIZED,
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_MINIMIZED,
+                               BOTH,
+                               popup_browser.get()));
+}
+
+// Test that the default show state override behavior is properly handled.
+TEST_F(WindowSizerTestWithBrowser, TestShowStateDefaults) {
+  // Creating a browser & window to play with.
+  scoped_ptr<aura::Window> window(
+      aura::test::CreateTestWindowWithId(0, NULL));
+  window->SetBounds(gfx::Rect(16, 32, 640, 320));
+
+  scoped_ptr<TestingProfile> profile(new TestingProfile());
+
+  scoped_ptr<BrowserWindow> browser_window(
+      new TestBrowserWindowAura(window.get()));
+  Browser::CreateParams window_params(Browser::TYPE_TABBED, profile.get());
+  window_params.window = browser_window.get();
+  scoped_ptr<Browser> browser(new Browser(window_params));
+
+  // Create also a popup browser since that behaves slightly different for
+  // defaults.
+  scoped_ptr<aura::Window> popup(
+      aura::test::CreateTestWindowWithId(1, NULL));
+  popup->SetBounds(gfx::Rect(16, 32, 128, 256));
+
+  scoped_ptr<BrowserWindow> browser_popup(
+      new TestBrowserWindowAura(popup.get()));
+  Browser::CreateParams popup_params(Browser::TYPE_POPUP, profile.get());
+  popup_params.window = browser_window.get();
+  scoped_ptr<Browser> popup_browser(new Browser(popup_params));
+
+  // Check that a browser creation state always get used if not given as
+  // SHOW_STATE_DEFAULT.
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_MAXIMIZED,
+                               DEFAULT,
+                               browser.get()), ui::SHOW_STATE_DEFAULT);
+  browser->set_initial_show_state(ui::SHOW_STATE_MINIMIZED);
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_MAXIMIZED,
+                               BOTH,
+                               browser.get()), ui::SHOW_STATE_MINIMIZED);
+  browser->set_initial_show_state(ui::SHOW_STATE_NORMAL);
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
+                               ui::SHOW_STATE_MAXIMIZED,
+                               BOTH,
+                               browser.get()), ui::SHOW_STATE_NORMAL);
+  browser->set_initial_show_state(ui::SHOW_STATE_MAXIMIZED);
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               browser.get()), ui::SHOW_STATE_MAXIMIZED);
+
+  // Check that setting the maximized command line option is forcing the
+  // maximized state.
+  CommandLine::ForCurrentProcess()->AppendSwitch(switches::kStartMaximized);
+
+  browser->set_initial_show_state(ui::SHOW_STATE_NORMAL);
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               browser.get()), ui::SHOW_STATE_MAXIMIZED);
+
+  // The popup should favor the initial show state over the command line.
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL,
+                               ui::SHOW_STATE_NORMAL,
+                               BOTH,
+                               popup_browser.get()), ui::SHOW_STATE_NORMAL);
+}