Open non browser window on the target display

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_ash.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_init.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 "content/public/test/render_view_test.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/aura/client/activation_client.h"
 #include "ui/aura/client/aura_constants.h"
 #include "ui/aura/env.h"
 #include "ui/aura/root_window.h"
@@ skipping 825 matching lines @@
                                browser_window->browser(),
                                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), ui::SHOW_STATE_NORMAL);
 }
+
+// Test that the target root window is used as the destionation of
+// the non browser window. This differ from PersistedBoundsCase
+// in that this uses real ash shell implementations + StateProvider
+// TargetDisplayProvider, rather than mocks.
+TEST_F(WindowSizerAshTest, DefaultBoundsInTargetDisplay) {
+  if (!SupportsMultipleDisplays() || !chrome::ShouldOpenAshOnStartup())
+    return;
+  UpdateDisplay("500x500,600x600");
+  {
+    aura::Window* first_root =
+        ash::Shell::GetAllRootWindows()[0];
+    ash::internal::ScopedTargetRootWindow tmp(
+        first_root);
+    gfx::Rect bounds;
+    ui::WindowShowState show_state;
+    WindowSizer::GetBrowserWindowBoundsAndShowState(
+        std::string(),
+        gfx::Rect(),
+        NULL,
+        &bounds,
+        &show_state);
+    EXPECT_TRUE(first_root->GetBoundsInScreen().Contains(bounds));
+  }
+  {
+    aura::Window* second_root =
+        ash::Shell::GetAllRootWindows()[1];
+    ash::internal::ScopedTargetRootWindow tmp(
+        second_root);
+    gfx::Rect bounds;
+    ui::WindowShowState show_state;
+    WindowSizer::GetBrowserWindowBoundsAndShowState(
+        std::string(),
+        gfx::Rect(),
+        NULL,
+        &bounds,
+        &show_state);
+    EXPECT_TRUE(second_root->GetBoundsInScreen().Contains(bounds));
+  }
+}