Aura: ShellDelegate GetCycleWindowList can request order by activity

alicet@ is working on window switching animations for compact mode. We're finding that while most-recently-used cycling works pretty well for normal mode it's pretty confusing for compact mode, since you can't see an overview of the windows.  This patch allows the window cycler controller to choose between most-recently-used and traditional linear order while we're sorting out which works better.

BUG=none
TEST=aura_shell_unittests WindowCycleControllerTest

Review URL: https://chromiumcodereview.appspot.com/9112044

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@117908 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 10 insertions, 3 deletions

diff --git a/ash/shell_delegate.h b/ash/shell_delegate.h
index 0bb1ce6..0f39756 100644
--- a/ash/shell_delegate.h
+++ b/ash/shell_delegate.h
@@ -32,6 +32,11 @@ struct LauncherItem;
 // Delegate of the Shell.
 class ASH_EXPORT ShellDelegate {
  public:
+  enum CycleOrder {
+    ORDER_MRU,  // Most recently used
+    ORDER_LINEAR
+  };
+
   // The Shell owns the delegate.
   virtual ~ShellDelegate() {}
 
@@ -50,10 +55,12 @@ class ASH_EXPORT ShellDelegate {
   virtual AppListViewDelegate* CreateAppListViewDelegate() = 0;
 
   // Returns a list of windows to cycle with keyboard shortcuts (e.g. alt-tab
-  // or the window switching key).  Windows should be in most-recently-used
-  // order with the currently active window at the front of the list.  The list
+  // or the window switching key).  If |order_by_activity| is true then windows
+  // are returned in most-recently-used order with the currently active window
+  // at the front of the list.  Otherwise any order may be returned.  The list
   // does not contain NULL pointers.
-  virtual std::vector<aura::Window*> GetCycleWindowList() const = 0;
+  virtual std::vector<aura::Window*> GetCycleWindowList(
+      CycleOrder order) const = 0;
 
   // Invoked when the user clicks on a window entry in the launcher.
   virtual void LauncherItemClicked(const LauncherItem& item) = 0;