Do not CrossFade the bounds change of AdjustWindowBoundsForWorkAreaChange()

ash/wm/workspace/workspace_layout_manager.cc

Index: ash/wm/workspace/workspace_layout_manager.cc
diff --git a/ash/wm/workspace/workspace_layout_manager.cc b/ash/wm/workspace/workspace_layout_manager.cc
index 9bf7e70f9e0c3dc1735884212c2fa518f900f198..3ea4d6c3786da2dba9355f455ab0d8678af4a4a5 100644
--- a/ash/wm/workspace/workspace_layout_manager.cc
+++ b/ash/wm/workspace/workspace_layout_manager.cc
@@ -213,16 +213,11 @@ void WorkspaceLayoutManager::AdjustWindowBoundsForWorkAreaChange(
   if (!GetTrackedByWorkspace(window))
     return;
 
-  // Use cross fade transition for the maximized window if the adjustment
-  // happens due to the shelf's visibility change. Otherwise the background
-  // can be seen slightly between the bottom edge of resized-window and
-  // the animating shelf.
-  // TODO(mukai): this cause slight blur at the window frame because of the
-  // cross fade. I think this is better, but should reconsider if someone
-  // raises voice for this.
+  // Do not cross fade here: the window's layer hierarchy may be messed up for
+  // the transition between mirroring and extended. See also: crbug.com/267698

[oshima, 2013/08/29 18:56:48]

can you add

TODO(oshima): Differentiate display change and shelf visibility change, and
bring back CrossFade animation.

[Jun Mukai, 2013/08/29 19:48:14]

Done.

   if (wm::IsWindowMaximized(window) &&
       reason == ADJUST_WINDOW_WORK_AREA_INSETS_CHANGED) {
-    CrossFadeToBounds(window, ScreenAsh::GetMaximizedWindowBoundsInParent(
+    SetChildBoundsDirect(window, ScreenAsh::GetMaximizedWindowBoundsInParent(
         window->parent()->parent()));
     return;
   }