Add window states docked; and docked minimized. Add wm window event to set docked and undocked.

ash/wm/dock/docked_window_layout_manager.cc

Index: ash/wm/dock/docked_window_layout_manager.cc
diff --git a/ash/wm/dock/docked_window_layout_manager.cc b/ash/wm/dock/docked_window_layout_manager.cc
index 14ccaa37329ea6a35b616c08f37d6c41097d51dd..8ab08941f352bad4c29f61a114779d68cea05fb1 100644
--- a/ash/wm/dock/docked_window_layout_manager.cc
+++ b/ash/wm/dock/docked_window_layout_manager.cc
@@ -576,16 +576,21 @@ DockedAlignment DockedWindowLayoutManager::GetAlignmentOfWindow(
 }
 
 DockedAlignment DockedWindowLayoutManager::CalculateAlignment() const {
-  // Find a child that is not being dragged and is not a popup.
+  return CalculateAlignmentExcept(dragged_window_);
+}
+
+DockedAlignment DockedWindowLayoutManager::CalculateAlignmentExcept(
+    const aura::Window* window) const {
+  // Find a child that is not the window being queried and is not a popup.
   // If such exists the current alignment is returned - even if some of the
   // children are hidden or minimized (so they can be restored without losing
   // the docked state).
   for (size_t i = 0; i < dock_container_->children().size(); ++i) {
-    aura::Window* window(dock_container_->children()[i]);
-    if (window != dragged_window_ && !IsPopupOrTransient(window))
+    aura::Window* child(dock_container_->children()[i]);
+    if (window != child && !IsPopupOrTransient(child))
       return alignment_;
   }
-  // No docked windows remain other than possibly the window being dragged.
+  // No docked windows remain other than possibly the window being queried.
   // Return |NONE| to indicate that windows may get docked on either side.
   return DOCKED_ALIGNMENT_NONE;
 }
@@ -620,25 +625,48 @@ bool DockedWindowLayoutManager::CanDockWindow(
   if (GetWindowHeightCloseTo(window, work_area.height()) > work_area.height())
     return false;
   // Cannot dock on the other size from an existing dock.
-  const DockedAlignment alignment = CalculateAlignment();
+  const DockedAlignment alignment = CalculateAlignmentExcept(window);
   if (desired_alignment != DOCKED_ALIGNMENT_NONE &&
       alignment != DOCKED_ALIGNMENT_NONE &&
       alignment != desired_alignment) {
     return false;
   }
   // Do not allow docking on the same side as shelf.
+  return IsDockedAlignmentValid(desired_alignment);
+}
+
+bool DockedWindowLayoutManager::IsDockedAlignmentValid(
+    DockedAlignment alignment) const {
   ShelfAlignment shelf_alignment = SHELF_ALIGNMENT_BOTTOM;

[oshima, 2014/10/01 23:48:57]

nit:

could you please change to
shelf_alignment = shelf_ ? shelf_->slignment() : XXX_BOTTOM?

[dtapuska, 2014/10/02 21:05:28]

Done.

   if (shelf_)
     shelf_alignment = shelf_->alignment();
-  if ((desired_alignment == DOCKED_ALIGNMENT_LEFT &&
+  if ((alignment == DOCKED_ALIGNMENT_LEFT &&
        shelf_alignment == SHELF_ALIGNMENT_LEFT) ||
-      (desired_alignment == DOCKED_ALIGNMENT_RIGHT &&
+      (alignment == DOCKED_ALIGNMENT_RIGHT &&
        shelf_alignment == SHELF_ALIGNMENT_RIGHT)) {
     return false;
   }
   return true;
 }
 
+void DockedWindowLayoutManager::MaybeSetDesiredDockedAlignment(
+    DockedAlignment alignment) {
+  // If the requested alignment is |NONE| or there are no
+  // docked windows return early as we can't change whether there is a
+  // dock or not. If the requested alignment is the same as the current
+  // alignment return early as an optimization.
+  if (alignment == DOCKED_ALIGNMENT_NONE ||
+      alignment_ == DOCKED_ALIGNMENT_NONE ||
+      alignment_ == alignment ||
+      !IsDockedAlignmentValid(alignment)) {
+    return;
+  }
+  alignment_ = alignment;
+
+  Relayout();
+  UpdateDockBounds(DockedWindowLayoutManagerObserver::CHILD_CHANGED);
+}
+
 void DockedWindowLayoutManager::OnShelfBoundsChanged() {
   Relayout();
   UpdateDockBounds(DockedWindowLayoutManagerObserver::DISPLAY_INSETS_CHANGED);
@@ -700,8 +728,10 @@ void DockedWindowLayoutManager::OnChildWindowVisibilityChanged(
     bool visible) {
   if (IsPopupOrTransient(child))
     return;
-  if (visible)
-    wm::GetWindowState(child)->Restore();
+
+  wm::WindowState* window_state = wm::GetWindowState(child);
+  if (visible && window_state->IsMinimized())

[oshima, 2014/10/01 23:48:57]

in which state (other than minimized) the window can be re-shown?

[dtapuska, 2014/10/02 21:05:28]

When restoring a app/panel (not a browser window); the window is restored first
by some other code; and if this code executes (ie; Restore()) it will undocks
the window; so you end up seeing that restoring panels doesn't work. So the
safeguard is that we only want to restore minimized windows here.

+    window_state->Restore();
   Relayout();
   UpdateDockBounds(DockedWindowLayoutManagerObserver::CHILD_CHANGED);
 }
@@ -815,15 +845,15 @@ void DockedWindowLayoutManager::OnPreWindowStateTypeChange(
   // until OnFullscreenStateChange is called when exiting fullscreen.
   if (in_fullscreen_)
     return;
-  if (window_state->IsMinimized()) {
-    MinimizeDockedWindow(window_state);
-  } else if (window_state->IsMaximizedOrFullscreen() ||
-             window_state->IsSnapped()) {
+  if (!window_state->IsDocked()) {
     if (window != dragged_window_) {
       UndockWindow(window);
-      RecordUmaAction(DOCKED_ACTION_MAXIMIZE, DOCKED_ACTION_SOURCE_UNKNOWN);
+      if (window_state->IsMaximizedOrFullscreen())
+        RecordUmaAction(DOCKED_ACTION_MAXIMIZE, DOCKED_ACTION_SOURCE_UNKNOWN);

[varkha, 2014/10/01 20:33:14]

Should probably record some other action in UMA (probably DOCKED_ACTION_UNDOCK)
when it is just undock and not DOCKED_ACTION_MAXIMIZE. Note that docked->snapped
state transition was recorded in UMA before as DOCKED_ACTION_MAXIMIZE action.
I am not sure if this belongs in this CL or the one that adds accelerators - up
to you.

[dtapuska, 2014/10/03 15:45:17]

Done.

     }
-  } else if (old_type == wm::WINDOW_STATE_TYPE_MINIMIZED) {
+  } else if (window_state->IsMinimized()) {
+    MinimizeDockedWindow(window_state);
+  } else if (old_type == wm::WINDOW_STATE_TYPE_DOCKED_MINIMIZED) {
     RestoreDockedWindow(window_state);
   }
 }
@@ -946,7 +976,7 @@ void DockedWindowLayoutManager::RestoreDockedWindow(
 
   // Evict the window if it can no longer be docked because of its height.
   if (!CanDockWindow(window, DOCKED_ALIGNMENT_NONE)) {
-    UndockWindow(window);
+    window_state->Restore();
     RecordUmaAction(DOCKED_ACTION_EVICT, DOCKED_ACTION_SOURCE_UNKNOWN);
     return;
   }