Put the logic to maximize/fullscreen window back in OnWindowAddedToLayout

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 efe6c6803514ec5f40897ba0f5c33cd045b9a1a8..0f9c8f7fdab3eef3fd09694b64a9eec228e6451a 100644
--- a/ash/wm/workspace/workspace_layout_manager.cc
+++ b/ash/wm/workspace/workspace_layout_manager.cc
@@ -122,6 +122,11 @@ void WorkspaceLayoutManager::OnWindowAddedToLayout(Window* child) {
   windows_.insert(child);
   child->AddObserver(this);
   window_state->AddObserver(this);
+  // TODO(oshima): This is necessary as the call in
+  // AdjustWindowBoundsWhenAdded is skipped when the bounds is
+  // empty. Investigate if we can eliminate this dup.
+  if (!window_state->is_dragged())
+    SetMaximizedOrFullscreenBounds(window_state);

[pkotwicz, 2014/02/14 00:22:26]

Optional Nit: You could modify the implementation of
AdjustWindowBoundsWhenAdded() given that it is only invoked from here

[oshima, 2014/02/14 00:42:49]

Thanks, I'll clean up in separate CL with tests coverage.

   UpdateShelfVisibility();
   UpdateFullscreenState();
   WindowPositioner::RearrangeVisibleWindowOnShow(child);