Only include normal type windows in the bounding box of an overview item.

BUG=448388
TEST=With 2 or more maximized windows, enter overview and position the mouse where an image link will move under during the transition. Then exit and reenter overview without moving the mouse. The two windows should still be the same size as each other.

Review URL: https://codereview.chromium.org/1384953005

Cr-Commit-Position: refs/heads/master@{#352724}

1 files changed, 6 insertions, 0 deletions

diff --git a/ash/wm/overview/scoped_transform_overview_window.cc b/ash/wm/overview/scoped_transform_overview_window.cc
index a3ebe83..993dcac 100644
--- a/ash/wm/overview/scoped_transform_overview_window.cc
+++ b/ash/wm/overview/scoped_transform_overview_window.cc
@@ -225,6 +225,12 @@ bool ScopedTransformOverviewWindow::Contains(const aura::Window* target) const {
 gfx::Rect ScopedTransformOverviewWindow::GetTargetBoundsInScreen() const {
   gfx::Rect bounds;
   for (const auto& window : GetTransientTreeIterator(window_)) {
+    // Ignore other window types when computing bounding box of window
+    // selector target item.
+    if (window != window_ && window->type() != ui::wm::WINDOW_TYPE_NORMAL &&
+        window->type() != ui::wm::WINDOW_TYPE_PANEL) {
+      continue;
+    }
     bounds.Union(ScreenUtil::ConvertRectToScreen(window->parent(),
                                                  window->GetTargetBounds()));
   }