Fix the browser match rules.

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 "ash/scoped_target_root_window.h"
 #include "ash/screen_util.h"
 #include "ash/shell.h"
 #include "ash/test/ash_test_base.h"
 #include "ash/test/test_shell_delegate.h"
 #include "ash/wm/window_positioner.h"
 #include "ash/wm/window_resizer.h"
 #include "ash/wm/window_state.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/ash/ash_util.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/test_browser_window_aura.h"
 #include "chrome/test/base/testing_profile.h"
 #include "content/public/test/render_view_test.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/env.h"
 #include "ui/aura/test/test_windows.h"
 #include "ui/aura/window_event_dispatcher.h"
 #include "ui/gfx/screen.h"
 #include "ui/wm/public/activation_client.h"
 
 typedef ash::test::AshTestBase WindowSizerAshTest;
 
 namespace {
 
-// 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)
-      : native_window_(native_window) {
-  }
-  ~TestBrowserWindowAura() override {}
-
-  // TestBrowserWindow overrides:
-  void Show() override {
-    native_window_->Show();
-    Activate();
-  }
-  void Hide() override { native_window_->Hide(); }
-  void Activate() override {
-    aura::client::GetActivationClient(
-        native_window_->GetRootWindow())->ActivateWindow(native_window_.get());
-  }
-  gfx::NativeWindow GetNativeWindow() const override {
-    return native_window_.get();
-  }
-  gfx::Rect GetBounds() const override { return native_window_->bounds(); }
-
-  Browser* browser() { return browser_.get(); }
-
-  void CreateBrowser(const Browser::CreateParams& params) {
-    Browser::CreateParams create_params = params;
-    create_params.window = this;
-    browser_.reset(new Browser(create_params));
-    if (browser_->is_type_tabbed() || browser_->is_app()) {
-      ash::wm::GetWindowState(native_window_.get())->
-          set_window_position_managed(true);
-    }
-  }
-
- private:
-  scoped_ptr<Browser> browser_;
-  scoped_ptr<aura::Window> native_window_;
-
-  DISALLOW_COPY_AND_ASSIGN(TestBrowserWindowAura);
-};
-
-scoped_ptr<TestBrowserWindowAura> CreateTestBrowserWindow(
-    aura::Window* window,
-    const gfx::Rect& bounds,
-    const Browser::CreateParams& params) {
+scoped_ptr<Browser> CreateTestBrowser(aura::Window* window,
+                                      const gfx::Rect& bounds,
+                                      Browser::CreateParams* params) {
   if (!bounds.IsEmpty())
     window->SetBounds(bounds);
-  scoped_ptr<TestBrowserWindowAura> browser_window(
-      new TestBrowserWindowAura(window));
-  browser_window->CreateBrowser(params);
-  return browser_window.Pass();
+  scoped_ptr<Browser> browser =
+      chrome::CreateBrowserWithAuraTestWindowForParams(make_scoped_ptr(window),
+                                                       params);
+  if (browser->is_type_tabbed() || browser->is_app()) {
+    ash::wm::GetWindowState(browser->window()->GetNativeWindow())
+        ->set_window_position_managed(true);
+  }
+  return browser.Pass();
 }
 
 }  // namespace
 
 // On desktop linux aura, we currently don't use the ash frame, breaking some
 // tests which expect ash sizes: http://crbug.com/303862
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #define MAYBE_DefaultSizeCase DISABLED_DefaultSizeCase
 #else
 #define MAYBE_DefaultSizeCase DefaultSizeCase
@@ skipping 372 matching lines @@
   scoped_ptr<TestingProfile> profile(new TestingProfile());
   // Creating a popup handler here to make sure it does not interfere with the
   // existing windows.
   Browser::CreateParams native_params(profile.get(),
                                       chrome::HOST_DESKTOP_TYPE_ASH);
   scoped_ptr<Browser> browser(
       chrome::CreateBrowserWithTestWindowForParams(&native_params));
 
   // Creating a popup handler here to make sure it does not interfere with the
   // existing windows.
-  scoped_ptr<BrowserWindow> browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(0),
-      gfx::Rect(16, 32, 640, 320),
-      Browser::CreateParams(profile.get(), chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params2(profile.get(), chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser2(CreateTestBrowser(
+      CreateTestWindowInShellWithId(0), gfx::Rect(16, 32, 640, 320), &params2));
+  BrowserWindow* browser_window = browser2->window();
 
   // Creating a popup to make sure it does not interfere with the positioning.
-  scoped_ptr<TestBrowserWindowAura> browser_popup(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(1),
-      gfx::Rect(16, 32, 128, 256),
-      Browser::CreateParams(Browser::TYPE_POPUP, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params_popup(Browser::TYPE_POPUP, profile.get(),
+                                     chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser_popup(
+      CreateTestBrowser(CreateTestWindowInShellWithId(1),
+                        gfx::Rect(16, 32, 128, 256), &params_popup));
 
   // Creating a panel to make sure it does not interfere with the positioning.
-  scoped_ptr<BrowserWindow> browser_panel(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(2),
-      gfx::Rect(32, 48, 256, 512),
-      Browser::CreateParams(Browser::TYPE_POPUP, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params_panel(Browser::TYPE_POPUP, profile.get(),
+                                     chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser_panel(
+      CreateTestBrowser(CreateTestWindowInShellWithId(2),
+                        gfx::Rect(32, 48, 256, 512), &params_panel));
+
   browser_window->Show();
   { // Make sure that popups do not get changed.
     gfx::Rect window_bounds;
     GetWindowBounds(p1600x1200, p1600x1200, gfx::Rect(),
-                    gfx::Rect(50, 100, 300, 150), bottom_s1600x1200,
-                    PERSISTED, browser_popup->browser(),
-                    gfx::Rect(), &window_bounds);
+                    gfx::Rect(50, 100, 300, 150), bottom_s1600x1200, PERSISTED,
+                    browser_popup.get(), gfx::Rect(), &window_bounds);
     EXPECT_EQ("50,100 300x150", window_bounds.ToString());
   }
 
   browser_window->Hide();
   { // If a window is there but not shown the persisted default should be used.
     gfx::Rect window_bounds;
     GetWindowBounds(p1600x1200, p1600x1200, gfx::Rect(),
                     gfx::Rect(50, 100, 300, 150), bottom_s1600x1200,
                     PERSISTED, browser.get(), gfx::Rect(), &window_bounds);
     EXPECT_EQ("50,100 300x150", window_bounds.ToString());
@@ skipping 113 matching lines @@
   gfx::Rect primary_bounds = ash::Shell::GetInstance()->GetScreen()->
       GetPrimaryDisplay().bounds();
   gfx::Rect secondary_bounds = ash::ScreenUtil::GetSecondaryDisplay().bounds();
 
   ash::Shell::GetInstance()->set_target_root_window(
       ash::Shell::GetPrimaryRootWindow());
 
   scoped_ptr<TestingProfile> profile(new TestingProfile());
 
   // Create browser windows that are used as reference.
-  scoped_ptr<BrowserWindow> browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(0),
-      gfx::Rect(10, 10, 200, 200),
-      Browser::CreateParams(profile.get(), chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params(profile.get(), chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser(CreateTestBrowser(
+      CreateTestWindowInShellWithId(0), gfx::Rect(10, 10, 200, 200), &params));
+  BrowserWindow* browser_window = browser->window();
+  gfx::NativeWindow native_window = browser_window->GetNativeWindow();
   browser_window->Show();
-  EXPECT_EQ(browser_window->GetNativeWindow()->GetRootWindow(),
-            ash::Shell::GetTargetRootWindow());
+  EXPECT_EQ(native_window->GetRootWindow(), ash::Shell::GetTargetRootWindow());
 
-  scoped_ptr<BrowserWindow> another_browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(1),
-      gfx::Rect(400, 10, 300, 300),
-      Browser::CreateParams(profile.get(), chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams another_params(profile.get(),
+                                       chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> another_browser(
+      CreateTestBrowser(CreateTestWindowInShellWithId(1),
+                        gfx::Rect(400, 10, 300, 300), &another_params));
+  BrowserWindow* another_browser_window = another_browser->window();
+  gfx::NativeWindow another_native_window =
+      another_browser_window->GetNativeWindow();
   another_browser_window->Show();
 
   // Creating a new window to verify the new placement.
-  scoped_ptr<TestBrowserWindowAura> new_browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(0),
-      gfx::Rect(),
-      Browser::CreateParams(profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams new_params(profile.get(),
+                                   chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> new_browser(CreateTestBrowser(
+      CreateTestWindowInShellWithId(0), gfx::Rect(), &new_params));
 
   // Make sure the primary root is active.
   ASSERT_EQ(ash::Shell::GetPrimaryRootWindow(),
             ash::Shell::GetTargetRootWindow());
 
   // First new window should be in the primary.
   {
     gfx::Rect window_bounds;
-    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds,
-                    gfx::Rect(), secondary_bounds,
-                    PERSISTED, new_browser_window->browser(),
-                    gfx::Rect(), &window_bounds);
+    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds, gfx::Rect(),
+                    secondary_bounds, PERSISTED, new_browser.get(), gfx::Rect(),
+                    &window_bounds);
     // TODO(oshima): Use exact bounds when the window_sizer_ash is
     // moved to ash and changed to include the result from
     // RearrangeVisibleWindowOnShow.
     EXPECT_TRUE(primary_bounds.Contains(window_bounds));
   }
 
   // Move the window to the right side of the secondary display and create a new
   // window. It should be opened then on the secondary display.
   {
     gfx::Display second_display = ash::Shell::GetScreen()->
         GetDisplayNearestPoint(gfx::Point(1600 + 100,10));
     browser_window->GetNativeWindow()->SetBoundsInScreen(
         gfx::Rect(secondary_bounds.CenterPoint().x() - 100, 10, 200, 200),
         second_display);
-    browser_window->Activate();
+    aura::client::GetActivationClient(native_window->GetRootWindow())
+        ->ActivateWindow(native_window);
     EXPECT_NE(ash::Shell::GetPrimaryRootWindow(),
               ash::Shell::GetTargetRootWindow());
     gfx::Rect window_bounds;
-    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds,
-                    gfx::Rect(), secondary_bounds,
-                    PERSISTED, new_browser_window->browser(),
-                    gfx::Rect(), &window_bounds);
+    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds, gfx::Rect(),
+                    secondary_bounds, PERSISTED, new_browser.get(), gfx::Rect(),
+                    &window_bounds);
     // TODO(oshima): Use exact bounds when the window_sizer_ash is
     // moved to ash and changed to include the result from
     // RearrangeVisibleWindowOnShow.
     EXPECT_TRUE(secondary_bounds.Contains(window_bounds));
   }
 
   // Activate another window in the primary display and create a new window.
   // It should be created in the primary display.
   {
-    another_browser_window->Activate();
+    aura::client::GetActivationClient(another_native_window->GetRootWindow())
+        ->ActivateWindow(another_native_window);
     EXPECT_EQ(ash::Shell::GetPrimaryRootWindow(),
               ash::Shell::GetTargetRootWindow());
 
     gfx::Rect window_bounds;
-    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds,
-                    gfx::Rect(), secondary_bounds,
-                    PERSISTED, new_browser_window->browser(),
-                    gfx::Rect(), &window_bounds);
+    GetWindowBounds(p1600x1200, p1600x1200, secondary_bounds, gfx::Rect(),
+                    secondary_bounds, PERSISTED, new_browser.get(), gfx::Rect(),
+                    &window_bounds);
     // TODO(oshima): Use exact bounds when the window_sizer_ash is
     // moved to ash and changed to include the result from
     // RearrangeVisibleWindowOnShow.
     EXPECT_TRUE(primary_bounds.Contains(window_bounds));
   }
 }
 
 // On desktop linux aura, we currently don't use the ash frame, breaking some
 // tests which expect ash sizes: http://crbug.com/303862
 #if defined(OS_LINUX) && !defined(OS_CHROMEOS)
 #define MAYBE_TestShowState DISABLED_TestShowState
 #else
 #define MAYBE_TestShowState TestShowState
 #endif
 
 // Test that the show state is properly returned for non default cases.
 TEST_F(WindowSizerAshTest, MAYBE_TestShowState) {
   scoped_ptr<TestingProfile> profile(new TestingProfile());
 
   // Creating a browser & window to play with.
-  scoped_ptr<TestBrowserWindowAura> browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(0),
-      gfx::Rect(16, 32, 640, 320),
-      Browser::CreateParams(Browser::TYPE_TABBED, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params(Browser::TYPE_TABBED, profile.get(),
+                               chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser(CreateTestBrowser(
+      CreateTestWindowInShellWithId(0), gfx::Rect(16, 32, 640, 320), &params));
 
   // Create also a popup browser since that behaves different.
-  scoped_ptr<TestBrowserWindowAura> browser_popup(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(1),
-      gfx::Rect(16, 32, 640, 320),
-      Browser::CreateParams(Browser::TYPE_POPUP, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params_popup(Browser::TYPE_POPUP, profile.get(),
+                                     chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser_popup(
+      CreateTestBrowser(CreateTestWindowInShellWithId(1),
+                        gfx::Rect(16, 32, 640, 320), &params_popup));
 
   // 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_window->browser(),
-                               p1600x1200,
-                               p1600x1200));
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_NORMAL,
+                               BOTH, browser.get(), p1600x1200, p1600x1200));
   // A window that is smaller than the whole work area is set to default state.
   EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
-            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               browser_window->browser(),
-                               p1280x1024,
-                               p1600x1200));
+            GetWindowShowState(ui::SHOW_STATE_DEFAULT, ui::SHOW_STATE_NORMAL,
+                               BOTH, browser.get(), p1280x1024, p1600x1200));
   // A window that is sized to occupy the whole work area is maximized.
   EXPECT_EQ(ui::SHOW_STATE_MAXIMIZED,
-            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200));
+            GetWindowShowState(ui::SHOW_STATE_DEFAULT, ui::SHOW_STATE_NORMAL,
+                               BOTH, browser.get(), p1600x1200, p1600x1200));
   // 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,
-                               browser_popup->browser(),
-                               p1600x1200,
-                               p1600x1200));
+  EXPECT_EQ(
+      ui::SHOW_STATE_MAXIMIZED,
+      GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_NORMAL, BOTH,
+                         browser_popup.get(), p1600x1200, p1600x1200));
   // 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,
-                               browser_popup->browser(),
-                               p1600x1200,
-                               p1600x1200));
+  EXPECT_EQ(
+      ui::SHOW_STATE_NORMAL,
+      GetWindowShowState(ui::SHOW_STATE_DEFAULT, ui::SHOW_STATE_NORMAL, BOTH,
+                         browser_popup.get(), p1600x1200, p1600x1200));
 
   // 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<TestBrowserWindowAura> browser_window2(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(3),
-      gfx::Rect(16, 32, 640, 320),
-      Browser::CreateParams(Browser::TYPE_TABBED, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params2(Browser::TYPE_TABBED, profile.get(),
+                                chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser2(CreateTestBrowser(
+      CreateTestWindowInShellWithId(3), gfx::Rect(16, 32, 640, 320), &params2));
 
   // 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,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200));
+            GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_DEFAULT,
+                               BOTH, browser.get(), p1600x1200, p1600x1200));
   // 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,
-                               browser_popup->browser(),
-                               p1600x1200,
-                               p1600x1200));
+  EXPECT_EQ(
+      ui::SHOW_STATE_MAXIMIZED,
+      GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_MINIMIZED,
+                         BOTH, browser_popup.get(), p1600x1200, p1600x1200));
 
   // In smaller screen resolutions we default to maximized if there is no other
   // window visible.
   int min_size = ash::WindowPositioner::GetForceMaximizedWidthLimit() / 2;
   if (min_size > 0) {
     const gfx::Rect tiny_screen(0, 0, min_size, min_size);
-    EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
-              GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
-                                 ui::SHOW_STATE_DEFAULT,
-                                 BOTH,
-                                 browser_window->browser(),
-                                 tiny_screen,
-                                 tiny_screen));
-    browser_window->Hide();
-    EXPECT_EQ(ui::SHOW_STATE_MAXIMIZED,
-              GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
-                                 ui::SHOW_STATE_DEFAULT,
-                                 BOTH,
-                                 browser_window2->browser(),
-                                 tiny_screen,
-                                 tiny_screen));
-
+    EXPECT_EQ(
+        ui::SHOW_STATE_DEFAULT,
+        GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_DEFAULT,
+                           BOTH, browser.get(), tiny_screen, tiny_screen));
+    browser->window()->Hide();
+    EXPECT_EQ(
+        ui::SHOW_STATE_MAXIMIZED,
+        GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_DEFAULT,
+                           BOTH, browser2.get(), tiny_screen, tiny_screen));
   }
 }
 
 // Test that the default show state override behavior is properly handled.
 TEST_F(WindowSizerAshTest, TestShowStateDefaults) {
   // Creating a browser & window to play with.
   scoped_ptr<TestingProfile> profile(new TestingProfile());
 
-  scoped_ptr<TestBrowserWindowAura> browser_window(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(0),
-      gfx::Rect(16, 32, 640, 320),
-      Browser::CreateParams(Browser::TYPE_TABBED, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params(Browser::TYPE_TABBED, profile.get(),
+                               chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser(CreateTestBrowser(
+      CreateTestWindowInShellWithId(0), gfx::Rect(16, 32, 640, 320), &params));
 
   // Create also a popup browser since that behaves slightly different for
   // defaults.
-  scoped_ptr<TestBrowserWindowAura> browser_popup(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(1),
-      gfx::Rect(16, 32, 128, 256),
-      Browser::CreateParams(Browser::TYPE_POPUP, profile.get(),
-                            chrome::HOST_DESKTOP_TYPE_ASH)));
+  Browser::CreateParams params_popup(Browser::TYPE_POPUP, profile.get(),
+                                     chrome::HOST_DESKTOP_TYPE_ASH);
+  scoped_ptr<Browser> browser_popup(
+      CreateTestBrowser(CreateTestWindowInShellWithId(1),
+                        gfx::Rect(16, 32, 128, 256), &params_popup));
 
   // Check that a browser creation state always get used if not given as
   // SHOW_STATE_DEFAULT. On Windows ASH it should be SHOW_STATE_MAXIMIZED.
   ui::WindowShowState window_show_state =
-      GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
-                         ui::SHOW_STATE_MAXIMIZED,
-                         DEFAULT,
-                         browser_window->browser(),
-                         p1600x1200,
-                         p1600x1200);
+      GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_MAXIMIZED,
+                         DEFAULT, browser.get(), p1600x1200, p1600x1200);
 #if defined(OS_WIN)
   EXPECT_EQ(window_show_state, ui::SHOW_STATE_MAXIMIZED);
 #else
   EXPECT_EQ(window_show_state, ui::SHOW_STATE_DEFAULT);
 #endif
 
-  browser_window->browser()->set_initial_show_state(ui::SHOW_STATE_MINIMIZED);
-  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
-                               ui::SHOW_STATE_MAXIMIZED,
-                               BOTH,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200), ui::SHOW_STATE_MINIMIZED);
-  browser_window->browser()->set_initial_show_state(ui::SHOW_STATE_NORMAL);
-  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_MAXIMIZED,
-                               ui::SHOW_STATE_MAXIMIZED,
-                               BOTH,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200), ui::SHOW_STATE_NORMAL);
-  browser_window->browser()->set_initial_show_state(ui::SHOW_STATE_MAXIMIZED);
-  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200), ui::SHOW_STATE_MAXIMIZED);
+  browser->set_initial_show_state(ui::SHOW_STATE_MINIMIZED);
+  EXPECT_EQ(
+      GetWindowShowState(ui::SHOW_STATE_MAXIMIZED, ui::SHOW_STATE_MAXIMIZED,
+                         BOTH, browser.get(), p1600x1200, p1600x1200),
+      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(), p1600x1200, p1600x1200),
+      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(), p1600x1200, p1600x1200),
+            ui::SHOW_STATE_MAXIMIZED);
 
   // Check that setting the maximized command line option is forcing the
   // maximized state.
   base::CommandLine::ForCurrentProcess()->AppendSwitch(
       switches::kStartMaximized);
 
-  browser_window->browser()->set_initial_show_state(ui::SHOW_STATE_NORMAL);
-  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               browser_window->browser(),
-                               p1600x1200,
-                               p1600x1200), ui::SHOW_STATE_MAXIMIZED);
+  browser->set_initial_show_state(ui::SHOW_STATE_NORMAL);
+  EXPECT_EQ(GetWindowShowState(ui::SHOW_STATE_NORMAL, ui::SHOW_STATE_NORMAL,
+                               BOTH, browser.get(), p1600x1200, p1600x1200),
+            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,
-                               browser_popup->browser(),
-                               p1600x1200,
-                               p1600x1200), ui::SHOW_STATE_NORMAL);
+  EXPECT_EQ(
+      GetWindowShowState(ui::SHOW_STATE_NORMAL, ui::SHOW_STATE_NORMAL, BOTH,
+                         browser_popup.get(), p1600x1200, p1600x1200),
+      ui::SHOW_STATE_NORMAL);
 }
 
 TEST_F(WindowSizerAshTest, DefaultStateBecomesMaximized) {
   // Create a browser to pass into the GetWindowBounds function.
   scoped_ptr<TestingProfile> profile(new TestingProfile());
   Browser::CreateParams native_params(profile.get(),
                                       chrome::HOST_DESKTOP_TYPE_ASH);
   scoped_ptr<Browser> browser(
       chrome::CreateBrowserWithTestWindowForParams(&native_params));
 
@@ skipping 59 matching lines @@
     EXPECT_TRUE(second_root->GetBoundsInScreen().Contains(bounds));
   }
 }
 
 TEST_F(WindowSizerAshTest, TrustedPopupBehavior) {
   scoped_ptr<TestingProfile> profile(new TestingProfile());
   Browser::CreateParams trusted_popup_create_params(
       Browser::TYPE_POPUP, profile.get(), chrome::HOST_DESKTOP_TYPE_ASH);
   trusted_popup_create_params.trusted_source = true;
 
-  scoped_ptr<TestBrowserWindowAura> trusted_popup(CreateTestBrowserWindow(
-      CreateTestWindowInShellWithId(1),
-      gfx::Rect(16, 32, 640, 320),
-      trusted_popup_create_params));
+  scoped_ptr<Browser> trusted_popup(CreateTestBrowser(
+      CreateTestWindowInShellWithId(1), gfx::Rect(16, 32, 640, 320),
+      &trusted_popup_create_params));
   // Trusted popup windows should follow the saved show state and ignore the
   // last show state.
-  EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
-            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               trusted_popup->browser(),
-                               p1280x1024,
-                               p1600x1200));
+  EXPECT_EQ(
+      ui::SHOW_STATE_DEFAULT,
+      GetWindowShowState(ui::SHOW_STATE_DEFAULT, ui::SHOW_STATE_NORMAL, BOTH,
+                         trusted_popup.get(), p1280x1024, p1600x1200));
   // A popup that is sized to occupy the whole work area has default state.
-  EXPECT_EQ(ui::SHOW_STATE_DEFAULT,
-            GetWindowShowState(ui::SHOW_STATE_DEFAULT,
-                               ui::SHOW_STATE_NORMAL,
-                               BOTH,
-                               trusted_popup->browser(),
-                               p1600x1200,
-                               p1600x1200));
+  EXPECT_EQ(
+      ui::SHOW_STATE_DEFAULT,
+      GetWindowShowState(ui::SHOW_STATE_DEFAULT, ui::SHOW_STATE_NORMAL, BOTH,
+                         trusted_popup.get(), p1600x1200, p1600x1200));
 }