Undocks window first before side-snapping bounds

ash/wm/workspace/workspace_layout_manager.h

 // 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.
 
 #ifndef ASH_WM_WORKSPACE_WORKSPACE_LAYOUT_MANAGER_H_
 #define ASH_WM_WORKSPACE_WORKSPACE_LAYOUT_MANAGER_H_
 
 #include <set>
 
 #include "ash/shell_observer.h"
@@ skipping 69 matching lines @@
 
   // Updates the bounds of the window for a show state change from
   // |last_show_state|.
   void UpdateBoundsFromShowState(wm::WindowState* window_state,
                                  ui::WindowShowState last_show_state);
 
   // If |window_state| is maximized or fullscreen the bounds of the
   // window are set and true is returned. Does nothing otherwise.
   bool SetMaximizedOrFullscreenBounds(wm::WindowState* window_state);
 
+  // Restores side-snapped state when window or work area bounds change.
+  void AdjustSnappedBounds(wm::WindowState* window_state, gfx::Rect* bounds);
+
+  // Animates window bounds to a new location when work area bounds change or

[pkotwicz, 2013/11/27 20:24:43]

How about "Animates the window bounds to |bounds|."

I do not think that mentioning when the method is invoked adds much. The comment
does not list all of the invokers either. (e.g. when a side snapped window
becomes restored)

[varkha, 2013/11/28 01:09:37]

Done.

+  // when window bounds change for a side-snapped window.
+  void SetChildBoundsAnimated(aura::Window* child, const gfx::Rect& bounds);
+
   internal::ShelfLayoutManager* shelf_;
   aura::Window* window_;
 
   // The work area. Cached to avoid unnecessarily moving windows during a
   // workspace switch.
   gfx::Rect work_area_in_parent_;
 
   DISALLOW_COPY_AND_ASSIGN(WorkspaceLayoutManager);
 };
 
 }  // namespace internal
 }  // namespace ash
 
 #endif  // ASH_WM_WORKSPACE_WORKSPACE_LAYOUT_MANAGER_H_