[ash-md] Prevents a crash when entering overview mode while dragging a window

ash/common/wm/overview/window_selector_item.cc

Index: ash/common/wm/overview/window_selector_item.cc
diff --git a/ash/common/wm/overview/window_selector_item.cc b/ash/common/wm/overview/window_selector_item.cc
index 9eada970ce3163e80b5114ed1d10b4d0cc42820a..c639777d50293e9fa11ae3404357d01814920adf 100644
--- a/ash/common/wm/overview/window_selector_item.cc
+++ b/ash/common/wm/overview/window_selector_item.cc
@@ -427,8 +427,15 @@ void WindowSelectorItem::Shutdown() {
     // gradual upon exiting the overview mode.
     WmWindow* label_window =
         WmLookup::Get()->GetWindowForWidget(window_label_.get());
-    label_window->GetParent()->StackChildAbove(label_window,
-                                               transform_window_.window());
+
+    // |label_window| was originally created in the same container as the
+    // |transform_window_| but when closing overview the |transform_window_|
+    // could have been reparented if a drag was active. Only change stacking
+    // if the windows still belong to the same container.
+    if (label_window->GetParent() == transform_window_.window()->GetParent()) {

[tdanderson, 2016/10/20 21:09:33]

For my own learning, in general must two windows in the same container have the
same parent (i.e., the container itself) or is that just true of this specific
situation?

[varkha, 2016/10/20 23:54:40]

Yes the two windows in the same container have the same parent (that container).
The problem here is that one of those two windows in this case (the real window
rather than the label that shows the title in overview) is reparented when
ending a drag. Ending the drag also ends overview mode because the window
hierarchy changes. So we get into shutdown but still assume that the title
window and the real window belong to the same container when they are not.

Note: while windows are dragged they get reparented to one of the top-level
containers (like a docked container) to allow them to be on top of any other
windows including docked or panels. At the end of the drag this is reverted.

+      label_window->GetParent()->StackChildAbove(label_window,
+                                                 transform_window_.window());
+    }
   }
   if (background_view_) {
     background_view_->OnItemRestored();