Index: ash/wm/workspace/workspace_event_handler_unittest.cc |
diff --git a/ash/wm/workspace/workspace_event_handler_unittest.cc b/ash/wm/workspace/workspace_event_handler_unittest.cc |
index 9f4d326ed5937b05d0f50f1d3991a03e90bb6ad8..843d13649a489f77c95c352907570aa85e8781b0 100644 |
--- a/ash/wm/workspace/workspace_event_handler_unittest.cc |
+++ b/ash/wm/workspace/workspace_event_handler_unittest.cc |
@@ -7,7 +7,7 @@ |
#include "ash/screen_ash.h" |
#include "ash/shell.h" |
#include "ash/test/ash_test_base.h" |
-#include "ash/wm/property_util.h" |
+#include "ash/wm/window_state.h" |
#include "ash/wm/window_util.h" |
#include "ash/wm/workspace_controller.h" |
#include "ash/wm/workspace_controller_test_helper.h" |
@@ -109,8 +109,10 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickSingleAxisResizeEdge) { |
EXPECT_EQ(restored_bounds.width(), bounds_in_screen.width()); |
EXPECT_EQ(work_area.y(), bounds_in_screen.y()); |
EXPECT_EQ(work_area.height(), bounds_in_screen.height()); |
+ |
+ wm::WindowState* window_state = wm::GetWindowState(window.get()); |
// Single-axis maximization is not considered real maximization. |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
// Restore. |
generator.DoubleClickLeftButton(); |
@@ -118,14 +120,14 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickSingleAxisResizeEdge) { |
EXPECT_EQ(restored_bounds.ToString(), bounds_in_screen.ToString()); |
// Note that it should not even be restored at this point, it should have |
// also cleared the restore rectangle. |
- EXPECT_EQ(NULL, GetRestoreBoundsInScreen(window.get())); |
+ EXPECT_FALSE(window_state->HasRestoreBounds()); |
// Double-click the top resize edge again to maximize vertically, then double |
// click again to restore. |
generator.DoubleClickLeftButton(); |
wd.set_window_component(HTCAPTION); |
generator.DoubleClickLeftButton(); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
bounds_in_screen = window->GetBoundsInScreen(); |
EXPECT_EQ(restored_bounds.ToString(), bounds_in_screen.ToString()); |
@@ -138,7 +140,7 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickSingleAxisResizeEdge) { |
EXPECT_EQ(work_area.x(), bounds_in_screen.x()); |
EXPECT_EQ(work_area.width(), bounds_in_screen.width()); |
// Single-axis maximization is not considered real maximization. |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
// Restore. |
wd.set_window_component(HTCAPTION); |
@@ -174,7 +176,7 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickSingleAxisResizeEdge) { |
EXPECT_EQ(restored_bounds.width(), bounds_in_screen.width()); |
EXPECT_EQ(work_area2.y(), bounds_in_screen.y()); |
EXPECT_EQ(work_area2.height(), bounds_in_screen.height()); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
// Restore. |
wd.set_window_component(HTCAPTION); |
@@ -189,7 +191,7 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickSingleAxisResizeEdge) { |
EXPECT_EQ(restored_bounds.height(), bounds_in_screen.height()); |
EXPECT_EQ(work_area2.x(), bounds_in_screen.x()); |
EXPECT_EQ(work_area2.width(), bounds_in_screen.width()); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
// Restore. |
wd.set_window_component(HTCAPTION); |
@@ -251,16 +253,18 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickCaptionTogglesMaximize) { |
CreateTestWindow(&wd, gfx::Rect(1, 2, 30, 40))); |
window->SetProperty(aura::client::kCanMaximizeKey, true); |
wd.set_window_component(HTCAPTION); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ |
+ wm::WindowState* window_state = wm::GetWindowState(window.get()); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
aura::RootWindow* root = Shell::GetPrimaryRootWindow(); |
aura::test::EventGenerator generator(root, window.get()); |
generator.DoubleClickLeftButton(); |
EXPECT_NE("1,2 30x40", window->bounds().ToString()); |
- EXPECT_TRUE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_TRUE(window_state->IsMaximized()); |
generator.DoubleClickLeftButton(); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
EXPECT_EQ("1,2 30x40", window->bounds().ToString()); |
// Double-clicking the middle button shouldn't toggle the maximized state. |
@@ -274,7 +278,7 @@ TEST_F(WorkspaceEventHandlerTest, DoubleClickCaptionTogglesMaximize) { |
ui::EF_IS_DOUBLE_CLICK); |
root->AsRootWindowHostDelegate()->OnHostMouseEvent(&release); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
EXPECT_EQ("1,2 30x40", window->bounds().ToString()); |
EXPECT_FALSE(observer.DidPropertyChange(aura::client::kShowStateKey)); |
} |
@@ -285,19 +289,21 @@ TEST_F(WorkspaceEventHandlerTest, DoubleTapCaptionTogglesMaximize) { |
scoped_ptr<aura::Window> window(CreateTestWindow(&wd, bounds)); |
window->SetProperty(aura::client::kCanMaximizeKey, true); |
wd.set_window_component(HTCAPTION); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ |
+ wm::WindowState* window_state = wm::GetWindowState(window.get()); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
aura::test::EventGenerator generator(Shell::GetPrimaryRootWindow(), |
window.get()); |
generator.GestureTapAt(gfx::Point(25, 25)); |
generator.GestureTapAt(gfx::Point(25, 25)); |
RunAllPendingInMessageLoop(); |
EXPECT_NE(bounds.ToString(), window->bounds().ToString()); |
- EXPECT_TRUE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_TRUE(window_state->IsMaximized()); |
generator.GestureTapAt(gfx::Point(5, 5)); |
generator.GestureTapAt(gfx::Point(10, 10)); |
- EXPECT_FALSE(wm::IsWindowMaximized(window.get())); |
+ EXPECT_FALSE(window_state->IsMaximized()); |
EXPECT_EQ(bounds.ToString(), window->bounds().ToString()); |
} |