Don't reposition if shutdown is in progress

ash/common/wm/overview/window_grid.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 "ash/common/wm/overview/window_grid.h"
 
 #include <algorithm>
 #include <functional>
 #include <set>
 #include <utility>
@@ skipping 438 matching lines @@
 }
 
 void WindowGrid::PrepareForOverview() {
   if (ash::MaterialDesignController::IsOverviewMaterial())
     InitShieldWidget();
   for (auto iter = window_list_.begin(); iter != window_list_.end(); ++iter)
     (*iter)->PrepareForOverview();
 }
 
 void WindowGrid::PositionWindowsMD(bool animate) {
-  if (window_list_.empty())
+  if (window_list_.empty() || window_selector_->shutdown())

[varkha, 2016/08/17 21:49:31]

Should this early exit be in PositionWindows() instead?

[oshima, 2016/08/17 22:00:26]

Done.

     return;
 
   const int kUnlimited = -1;
   const size_t windows_count = window_list_.size();
   const base::CommandLine* command_line =
       base::CommandLine::ForCurrentProcess();
   int windows_to_use_masks = kMaxWindowsCountToHideHeaderWithMasks;
   if (command_line->HasSwitch(switches::kAshMaxWindowsToUseMaskInOverview) &&
       (!base::StringToInt(command_line->GetSwitchValueASCII(
                               switches::kAshMaxWindowsToUseMaskInOverview),
@@ skipping 578 matching lines @@
         *min_right = left;
       if (*max_right < left)
         *max_right = left;
     }
     *max_bottom = top + height;
   }
   return windows_fit;
 }
 
 }  // namespace ash