Undocks window first before side-snapping bounds

ash/wm/workspace_controller_unittest.cc

Index: ash/wm/workspace_controller_unittest.cc
diff --git a/ash/wm/workspace_controller_unittest.cc b/ash/wm/workspace_controller_unittest.cc
index 48e08fc41545760b62793cdd60fd8e9361c1829d..3048a8185279c92d43a2bcb9bd2c196026e92214 100644
--- a/ash/wm/workspace_controller_unittest.cc
+++ b/ash/wm/workspace_controller_unittest.cc
@@ -851,7 +851,7 @@ TEST_F(WorkspaceControllerTest, BasicAutoPlacingOnShowHide) {
 
   // Test4: A single manageable window should get centered.
   window4.reset();
-  window1_state->set_bounds_changed_by_user(false);
+  window1_state->SetBoundsChangedByUser(false);
   // Trigger the auto window placement function by showing (and hiding) it.
   window1->Hide();
   window1->Show();
@@ -881,7 +881,7 @@ TEST_F(WorkspaceControllerTest, TestUserMovedWindowRepositioning) {
 
   // Check that the current location gets preserved if the user has
   // positioned it previously.
-  window1_state->set_bounds_changed_by_user(true);
+  window1_state->SetBoundsChangedByUser(true);
   window1->Show();
   EXPECT_EQ("16,32 640x320", window1->bounds().ToString());
   // Flag should be still set.
@@ -902,7 +902,7 @@ TEST_F(WorkspaceControllerTest, TestUserMovedWindowRepositioning) {
   EXPECT_FALSE(window2_state->bounds_changed_by_user());
 
   // Going back to one shown window should keep the state.
-  window1_state->set_bounds_changed_by_user(true);
+  window1_state->SetBoundsChangedByUser(true);
   window2->Hide();
   EXPECT_EQ("0,32 640x320", window1->bounds().ToString());
   EXPECT_TRUE(window1_state->bounds_changed_by_user());