diff options
| author | yuzo@chromium.org <yuzo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-23 08:11:28 +0000 |
|---|---|---|
| committer | yuzo@chromium.org <yuzo@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-06-23 08:11:28 +0000 |
| commit | c0ba0da8a6c0d0cd33acf46441b0d2edc63b16c4 (patch) | |
| tree | 3fe96b3360d076c35544282145c7a1a71be6442a /chrome/browser | |
| parent | f6555adcd5160d011ea1dc613fa0387dcddd0b6b (diff) | |
| download | chromium_src-c0ba0da8a6c0d0cd33acf46441b0d2edc63b16c4.zip chromium_src-c0ba0da8a6c0d0cd33acf46441b0d2edc63b16c4.tar.gz chromium_src-c0ba0da8a6c0d0cd33acf46441b0d2edc63b16c4.tar.bz2 | |
For consistency, stop using the workarea coordinate in
saving/retrieving/adjusting window positions.
TEST=Open Chrome windows and see if they are placed properly.
TESTED=gcl try, manually
BUG=none
Review URL: http://codereview.chromium.org/141039
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@19012 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser')
| -rw-r--r-- | chrome/browser/window_sizer.cc | 2 | ||||
| -rw-r--r-- | chrome/browser/window_sizer_unittest.cc | 4 |
2 files changed, 2 insertions, 4 deletions
diff --git a/chrome/browser/window_sizer.cc b/chrome/browser/window_sizer.cc index 59eed33..10e3cb9 100644 --- a/chrome/browser/window_sizer.cc +++ b/chrome/browser/window_sizer.cc @@ -151,7 +151,6 @@ bool WindowSizer::GetLastWindowBounds(gfx::Rect* bounds) const { if (!state_provider_ || !state_provider_->GetLastActiveWindowState(bounds)) return false; gfx::Rect last_window_bounds = *bounds; - bounds->Offset(monitor_info_provider_->GetBoundsOffsetMatching(*bounds)); bounds->Offset(kWindowTilePixels, kWindowTilePixels); AdjustBoundsToBeVisibleOnMonitorContaining(last_window_bounds, bounds); return true; @@ -163,7 +162,6 @@ bool WindowSizer::GetSavedWindowBounds(gfx::Rect* bounds, if (!state_provider_ || !state_provider_->GetPersistentState(bounds, maximized)) return false; - bounds->Offset(monitor_info_provider_->GetBoundsOffsetMatching(*bounds)); AdjustBoundsToBeVisibleOnMonitorContaining(*bounds, bounds); return true; } diff --git a/chrome/browser/window_sizer_unittest.cc b/chrome/browser/window_sizer_unittest.cc index 552a340..a3e5ecd9 100644 --- a/chrome/browser/window_sizer_unittest.cc +++ b/chrome/browser/window_sizer_unittest.cc @@ -268,7 +268,7 @@ TEST(WindowSizerTest, LastWindowBoundsCase) { gfx::Rect(10, 10, 500, 400), false, LAST_ACTIVE, &window_bounds, &maximized); EXPECT_FALSE(maximized); - EXPECT_EQ(gfx::Rect(127, 20, 500, 400), window_bounds); + EXPECT_EQ(gfx::Rect(20, 20, 500, 400), window_bounds); } { // taskbar on top. @@ -278,7 +278,7 @@ TEST(WindowSizerTest, LastWindowBoundsCase) { gfx::Rect(10, 10, 500, 400), false, LAST_ACTIVE, &window_bounds, &maximized); EXPECT_FALSE(maximized); - EXPECT_EQ(gfx::Rect(20, 54, 500, 400), window_bounds); + EXPECT_EQ(gfx::Rect(20, 20, 500, 400), window_bounds); } { // too small to satisify the minimum visibility condition. |