Don't treat minimized windows as lower priority in MRU list (for Alt+Tab

ash/common/wm/mru_window_tracker.cc

 // Copyright 2013 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/mru_window_tracker.h"
 
 #include <algorithm>
 
 #include "ash/common/shell_window_ids.h"
 #include "ash/common/wm/focus_rules.h"
@@ skipping 16 matching lines @@
 // Adds the windows that can be cycled through for the specified window id to
 // |windows|.
 void AddTrackedWindows(WmWindow* root,
                        int container_id,
                        MruWindowTracker::WindowList* windows) {
   WmWindow* container = root->GetChildByShellWindowId(container_id);
   const MruWindowTracker::WindowList children(container->GetChildren());
   windows->insert(windows->end(), children.begin(), children.end());
 }
 
-// Returns whether |w1| should be considered less recently used than |w2|. This
-// is used for a stable sort to move minimized windows to the LRU end of the
-// list.
-bool CompareWindowState(WmWindow* w1, WmWindow* w2) {
-  return w1->GetWindowState()->IsMinimized() &&
-         !w2->GetWindowState()->IsMinimized();
-}
-
 // Returns a list of windows ordered by their stacking order.
 // If |mru_windows| is passed, these windows are moved to the front of the list.
 // It uses the given |should_include_window_predicate| to determine whether to
 // include a window in the returned list or not.
 MruWindowTracker::WindowList BuildWindowListInternal(
     const std::list<WmWindow*>* mru_windows,
     const CanActivateWindowPredicate& should_include_window_predicate) {
   MruWindowTracker::WindowList windows;
   WmWindow* active_root = WmShell::Get()->GetRootWindowForNewWindows();
   for (WmWindow* window : WmShell::Get()->GetAllRootWindows()) {
@@ skipping 33 matching lines @@
 
       MruWindowTracker::WindowList::iterator window =
           std::find(windows.begin(), windows.end(), *ix);
       if (window != windows.end()) {
         windows.erase(window);
         windows.push_back(*ix);
       }
     }
   }
 
-  // Move minimized windows to the beginning (LRU end) of the list.
-  std::stable_sort(windows.begin(), windows.end(), CompareWindowState);
-
   // Window cycling expects the topmost window at the front of the list.
   std::reverse(windows.begin(), windows.end());
 
   return windows;
 }
 
 }  // namespace
 
 //////////////////////////////////////////////////////////////////////////////
 // MruWindowTracker, public:
@@ skipping 52 matching lines @@
 
 void MruWindowTracker::OnWindowDestroyed(WmWindow* window) {
   // It's possible for OnWindowActivated() to be called after
   // OnWindowDestroying(). This means we need to override OnWindowDestroyed()
   // else we may end up with a deleted window in |mru_windows_|.
   mru_windows_.remove(window);
   window->RemoveObserver(this);
 }
 
 }  // namespace ash