Check that child window was not destroyed inside UpdateTransientChildVisibility.

ui/wm/core/transient_window_manager.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "ui/wm/core/transient_window_manager.h"
 
 #include <algorithm>
 #include <functional>
 
 #include "base/auto_reset.h"
@@ skipping 144 matching lines @@
   }
 }
 
 void TransientWindowManager::OnWindowVisibilityChanged(Window* window,
                                                        bool visible) {
   if (window_ != window)
     return;
 
   // If the window has transient children, updates the transient children's
   // visiblity as well.
-  for (Window* child : transient_children_)
-    Get(child)->UpdateTransientChildVisibility(visible);
+  Windows transient_children = transient_children_;
+  for (Window* child : transient_children) {
+    // Child window can be removed inside UpdateTransientChildVisibility call.
+    if (std::find(transient_children_.begin(), transient_children_.end(),
+                  child) != transient_children_.end())

[oshima, 2015/11/19 18:05:40]

you need {} in this case

+      Get(child)->UpdateTransientChildVisibility(visible);
+  }
 
   // Remember the show request in |show_on_parent_visible_| and hide it again
   // if the following conditions are met
   // - |parent_controls_visibility| is set to true.
   // - the window is hidden while the transient parent is not visible.
   // - Show/Hide was NOT called from TransientWindowManager.
   if (ignore_visibility_changed_event_ ||
       !transient_parent_ || !parent_controls_visibility_) {
     return;
   }
@@ skipping 36 matching lines @@
 
   // Destroy transient children, only after we've removed ourselves from our
   // parent, as destroying an active transient child may otherwise attempt to
   // refocus us.
   Windows transient_children(transient_children_);
   STLDeleteElements(&transient_children);
   DCHECK(transient_children_.empty());
 }
 
 }  // namespace wm