AdjustBoundsToEnsureMinimumWindowVisibility may cause unwanted shift if window dimension is small

ash/common/wm/default_state.cc

Index: ash/common/wm/default_state.cc
diff --git a/ash/common/wm/default_state.cc b/ash/common/wm/default_state.cc
index e7aa9e21265f821e039772f6541ea056e37a9cbb..415b55c0bbfed733111d2122106f164cd6bb0b01 100644
--- a/ash/common/wm/default_state.cc
+++ b/ash/common/wm/default_state.cc
@@ -476,7 +476,8 @@ bool DefaultState::ProcessWorkspaceEvents(WindowState* window_state,
       wm::AdjustBoundsToEnsureMinimumWindowVisibility(work_area_in_parent,
                                                       &bounds);
       window_state->AdjustSnappedBounds(&bounds);
-      if (window_state->window()->GetBounds() != bounds)
+      if (window_state->window()->GetBounds() != bounds &&
+          !window_state->ignored_by_shelf())

[sky, 2016/08/25 15:19:30]

ignored_by_shelf means:

  // True if the window is ignored by the shelf layout manager for
  // purposes of darkening the shelf.

You're changing the meaning to include don't adjust bounds on workspace area
changed. I don't think ignored_by_shelf should mean these two things.

I see two options:
1. ash shouldn't change the bounds of anything on a resize. This way client code
can do what it wants and we don't have race conditions. 
2. Add a property with the meaning of what you want, don't resize on workspace
change.
3. Only run this code on windows of type WINDOW_TYPE_NORMAL.

3 seems right to me, but there may be subtlety.

         window_state->SetBoundsDirectAnimated(bounds);
       return true;
     }