Reorder browser list on workspace switch

chrome/browser/ui/browser_list.h

 // Copyright (c) 2012 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 CHROME_BROWSER_UI_BROWSER_LIST_H_
 #define CHROME_BROWSER_UI_BROWSER_LIST_H_
 
 #include <stddef.h>
 
 #include <vector>
@@ skipping 51 matching lines @@
   // so notify and THEN delete the object.
   static void AddBrowser(Browser* browser);
   static void RemoveBrowser(Browser* browser);
 
   // Adds and removes |observer| from the observer list for all desktops.
   // Observers are responsible for making sure the notifying browser is relevant
   // to them (e.g., on the specific desktop they care about if any).
   static void AddObserver(chrome::BrowserListObserver* observer);
   static void RemoveObserver(chrome::BrowserListObserver* observer);
 
+  // Moves all the browsers that show on workspace |new_workspace| to the end of
+  // the browser list (i.e. the browsers that were "activated" most recently).
+  static void PartitionByWorkspace(const std::string& new_workspace);

[sky, 2016/07/07 21:31:11]

nit: IMO partition is obscure and while it matches the function you're using it
doesn't really indicate what this does. How about
MoveBrowsersInWorkspaceToFront? I realize front is a bit confusing here, but it
matches begin_last_active(), which returns the end.

+
   // Called by Browser objects when their window is activated (focused).  This
   // allows us to determine what the last active Browser was on each desktop.
   static void SetLastActive(Browser* browser);
 
   // Closes all browsers for |profile| across all desktops.
   // TODO(mlerman): Move the Profile Deletion flow to use the overloaded
   // version of this method with a callback, then remove this method.
   static void CloseAllBrowsersWithProfile(Profile* profile);
 
   // Closes all browsers for |profile| across all desktops. Uses
@@ skipping 51 matching lines @@
   // removal across all BrowserLists.
   static base::LazyInstance<
       base::ObserverList<chrome::BrowserListObserver>>::Leaky observers_;
 
   static BrowserList* instance_;
 
   DISALLOW_COPY_AND_ASSIGN(BrowserList);
 };
 
 #endif  // CHROME_BROWSER_UI_BROWSER_LIST_H_