Undocks window first before side-snapping bounds

ash/wm/workspace/workspace_layout_manager.h

Index: ash/wm/workspace/workspace_layout_manager.h
diff --git a/ash/wm/workspace/workspace_layout_manager.h b/ash/wm/workspace/workspace_layout_manager.h
index 30a88f2e4afcddfe74be94ad2c82473faec96af7..8e679929008a4590b92533b691f7cb57e58ce6e6 100644
--- a/ash/wm/workspace/workspace_layout_manager.h
+++ b/ash/wm/workspace/workspace_layout_manager.h
@@ -87,6 +87,13 @@ class ASH_EXPORT WorkspaceLayoutManager : public BaseLayoutManager {
   // 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_;