Chromium Code Reviews| Index: ash/accelerators/accelerator_controller_unittest.cc |
| diff --git a/ash/accelerators/accelerator_controller_unittest.cc b/ash/accelerators/accelerator_controller_unittest.cc |
| index 4c00f72734149fb158380a5edab6b0ca8579c21d..cf796890390ae571bdeea573c1087ea87ae75d85 100644 |
| --- a/ash/accelerators/accelerator_controller_unittest.cc |
| +++ b/ash/accelerators/accelerator_controller_unittest.cc |
| @@ -501,28 +501,12 @@ TEST_F(AcceleratorControllerTest, WindowSnap) { |
| wm::ActivateWindow(window.get()); |
| { |
| + gfx::Rect window_bounds = window->bounds(); |
| GetController()->PerformAction(WINDOW_SNAP_LEFT, dummy); |
| - gfx::Rect snap_left = window->bounds(); |
| - GetController()->PerformAction(WINDOW_SNAP_LEFT, dummy); |
| - EXPECT_NE(window->bounds().ToString(), snap_left.ToString()); |
| - GetController()->PerformAction(WINDOW_SNAP_LEFT, dummy); |
| - EXPECT_NE(window->bounds().ToString(), snap_left.ToString()); |
| - |
| - // It should cycle back to the first snapped position. |
| - GetController()->PerformAction(WINDOW_SNAP_LEFT, dummy); |
| - EXPECT_EQ(window->bounds().ToString(), snap_left.ToString()); |
| - } |
| - { |
| - GetController()->PerformAction(WINDOW_SNAP_RIGHT, dummy); |
| - gfx::Rect snap_right = window->bounds(); |
| - GetController()->PerformAction(WINDOW_SNAP_RIGHT, dummy); |
| - EXPECT_NE(window->bounds().ToString(), snap_right.ToString()); |
| - GetController()->PerformAction(WINDOW_SNAP_RIGHT, dummy); |
| - EXPECT_NE(window->bounds().ToString(), snap_right.ToString()); |
| - |
| - // It should cycle back to the first snapped position. |
| + EXPECT_NE(window->bounds().ToString(), window_bounds.ToString()); |
|
Mr4D (OOO till 08-26)
2013/08/29 01:03:15
Could you do some more tests towards the result (e
|
| + window_bounds = window->bounds(); |
| GetController()->PerformAction(WINDOW_SNAP_RIGHT, dummy); |
| - EXPECT_EQ(window->bounds().ToString(), snap_right.ToString()); |
| + EXPECT_NE(window->bounds().ToString(), window_bounds.ToString()); |
|
Mr4D (OOO till 08-26)
2013/08/29 01:03:15
Same here.
|
| } |
| { |
| gfx::Rect normal_bounds = window->bounds(); |