Improving window auto management between workspaces

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_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 "ash/wm/window_util.h"
 #include "base/compiler_specific.h"
 #include "chrome/browser/ui/browser.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 "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"
@@ skipping 467 matching lines @@
   scoped_ptr<Browser> panel_owning_browser(new Browser(panel_params));
 
   window->Show();
   { // With a shown window it's size should get returned.
     gfx::Rect window_bounds;
     GetWindowBounds(tentwentyfour, tentwentyfour, gfx::Rect(),
                     gfx::Rect(50, 100, 300, 150), bottom_nonprimary,
                     PERSISTED, browser.get(), gfx::Rect(), &window_bounds);
     // The position should be right flush.
     EXPECT_EQ("384,32 640x320", window_bounds.ToString());
-    // In addition the other window should have moved left flush.
-    EXPECT_EQ("0,32 640x320", window->bounds().ToString());
   }
 
   { // With the window shown - but more on the right side then on the left
     // side (and partially out of the screen), it should default to the other
     // side and inside the screen.
     window->SetBounds(gfx::Rect(350, 600, 640, 320));
     gfx::Rect window_bounds;
     GetWindowBounds(tentwentyfour, tentwentyfour, gfx::Rect(),
                     gfx::Rect(50, 100, 300, 150), bottom_nonprimary,
                     PERSISTED, browser.get(), gfx::Rect(), &window_bounds);
     // The position should be left & bottom flush.
     EXPECT_EQ("0,448 640x320", window_bounds.ToString());
-    // In addition the other window should have moved right flush.
-    EXPECT_EQ("384,600 640x320", window->bounds().ToString());
   }
 
   { // If the second windows right side is already over the right side of the
     // screen, it will not move back into the screen.
     window->SetBounds(gfx::Rect(1000, 600, 640, 320));
     gfx::Rect window_bounds;
     GetWindowBounds(tentwentyfour, tentwentyfour, gfx::Rect(),
                     gfx::Rect(50, 100, 300, 150), bottom_nonprimary,
                     PERSISTED, browser.get(), gfx::Rect(), &window_bounds);
     // The position should be left & bottom flush.
@@ skipping 170 matching lines @@
                                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);
 }