Refactor most recently used window tracking into a separate class.

ash/wm/mru_window_tracker.h

+// Copyright (c) 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.
+
+#ifndef ASH_WM_MRU_WINDOW_TRACKER_H_
+#define ASH_WM_MRU_WINDOW_TRACKER_H_
+
+#include <list>
+#include <vector>
+
+#include "ash/ash_export.h"
+#include "base/basictypes.h"
+#include "base/memory/scoped_ptr.h"
+#include "ui/aura/client/activation_change_observer.h"
+#include "ui/aura/window_observer.h"
+
+namespace aura {
+class RootWindow;
+class Window;
+namespace client {
+class ActivationClient;
+}
+}
+
+namespace ash {
+
+// Maintains a most recently used list of windows. This is used for window
+// cycling using Alt+Tab and overview mode.
+class ASH_EXPORT MruWindowTracker
+    : public aura::client::ActivationChangeObserver,
+      public aura::WindowObserver {
+ public:
+  typedef std::vector<aura::Window*> WindowList;
+
+  explicit MruWindowTracker(
+      aura::client::ActivationClient* activation_client);
+  virtual ~MruWindowTracker();
+
+  // Set up the observers to handle window changes for the containers we care
+  // about.  Called when a new root window is added.
+  void OnRootWindowAdded(aura::RootWindow* root_window);
+
+  // Returns the set of windows which can be cycled through. This method creates
+  // the vector based on the current set of windows across all valid root
+  // windows. As a result it is not necessarily the same as the set of
+  // windows being iterated over.
+  // If |mru_windows| is not NULL, windows in this list are put at the head of
+  // the window list.
+  // If |top_most_at_end| the window list will return in ascending order instead
+  // of the default descending.

[sky, 2013/07/29 15:31:05]

It isn't clear what descending/ascending means here. Document it.

[flackr, 2013/07/29 22:49:11]

Done.

+  static WindowList BuildWindowList(
+      const std::list<aura::Window*>* mru_windows,

[sky, 2013/07/29 15:31:05]

Why is mru_windows part of the API? Seems like its best done by the caller.

[flackr, 2013/07/29 22:49:11]

From a codesearch, the only callers who actually care about the order are
methods in window_cycle_controller. LinearCycle uses the stacking order while
alt-tab uses MRU order. The difference is that alt-tab currently activates every
window as you pass it and brings it to the front as you cycle. This is the
reason we maintain the MRU window list.

I've removed mru_windows from the static function, however, I think it would be
fine to just have one function which builds and use the Mru window list in all
cases since the callers that don't expect Mru order don't care about the order
(except for HandleLinearCycle in WindowCycleController which explicitly uses a
LRU list to ensure that you can single step through all windows).

+      bool top_most_at_end);
+
+  // Returns the set of windows which can be cycled through using the tracked
+  // list of most recently used windows. If |top_most_at_end| the window list

[sky, 2013/07/29 15:31:05]

Since you refer the ordering twice (here and BuildWindowList), maybe it should
be documented at the class level.

+  // is returned in ascending order.
+  WindowList BuildMruWindowList(bool top_most_at_end);
+
+  // Starts or stops ignoring window activations. If no longer ignoring
+  // activations the currently active window is moved to the front of the
+  // MRU window list. Used by WindowCycleList to avoid adding all cycled
+  // windows to the front of the MRU window list.
+  void IgnoreActivations(bool ignore);

[sky, 2013/07/29 15:31:05]

SetIgnoreActivations. That said, since you're going to have two classes using
this it feels rather brittle to have it a simple boolean. Should it be a
counter?

[flackr, 2013/07/29 22:49:11]

Done.

Overview mode won't actually need to ignore activations since it doesn't need to
activate windows to make them visible. Window cycler makes windows active as you
alt tab through them. I can still change this to a counter if you think it would
be safer.

+ private:

[sky, 2013/07/29 15:31:05]

nit: newline between 64/65.

[flackr, 2013/07/29 22:49:11]

Done.

+  // Checks if the window represents a container whose children we track.
+  static bool IsTrackedContainer(aura::Window* window);
+
+  // Overridden from aura::client::ActivationChangeObserver:
+  virtual void OnWindowActivated(aura::Window* gained_active,
+                                 aura::Window* lost_active) OVERRIDE;
+
+  // Overridden from WindowObserver:
+  virtual void OnWindowAdded(aura::Window* window) OVERRIDE;
+  virtual void OnWillRemoveWindow(aura::Window* window) OVERRIDE;
+  virtual void OnWindowDestroying(aura::Window* window) OVERRIDE;
+
+  // List of windows that have been activated in containers that we cycle
+  // through, sorted by most recently used.
+  std::list<aura::Window*> mru_windows_;
+
+  aura::client::ActivationClient* activation_client_;
+
+  bool ignore_window_activations_;
+
+  DISALLOW_COPY_AND_ASSIGN(MruWindowTracker);
+};
+
+}  // namespace ash
+
+#endif  // ASH_WM_MRU_WINDOW_TRACKER_H_