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>
 
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "base/observer_list.h"
+#include "ui/display/desktop_observer.h"
 
 class Browser;
 class Profile;
 
 namespace base {
 class FilePath;
 }
 
 namespace chrome {
 class BrowserListObserver;
 }
 
 // Maintains a list of Browser objects.
-class BrowserList {
+class BrowserList : private display::desktop::DesktopObserver {

[Lei Zhang, 2016/06/29 20:51:01]

This stays public, but you can leave the overrides in the private section below.
https://google.github.io/styleguide/cppguide.html#Inheritance

[Tom (Use chromium acct), 2016/06/29 21:12:50]

Done.

  public:
   typedef std::vector<Browser*> BrowserVector;
   typedef BrowserVector::const_iterator const_iterator;
   typedef BrowserVector::const_reverse_iterator const_reverse_iterator;
 
   // Returns the last active browser for this list.
   Browser* GetLastActive() const;
 
   // Browsers are added to the list before they have constructed windows,
   // so the |window()| member function may return NULL.
@@ skipping 50 matching lines @@
   // Returns true if at least one incognito session is active across all
   // desktops.
   static bool IsOffTheRecordSessionActive();
 
   // Returns true if at least one incognito session is active for |profile|
   // across all desktops.
   static bool IsOffTheRecordSessionActiveForProfile(Profile* profile);
 
  private:
   BrowserList();
-  ~BrowserList();
+  ~BrowserList() override;
+
+  // display::DesktopObserver:
+  void OnWorkspaceChanged(const std::string& new_workspace) override;
 
   // Helper method to remove a browser instance from a list of browsers
   static void RemoveBrowserFrom(Browser* browser, BrowserVector* browser_list);
 
   // Attempts to close |browsers_to_close| while respecting OnBeforeUnload
   // events. If there are no OnBeforeUnload events to be called,
   // |on_close_confirmed| will be called, with a parameter of |profile_path|,
   // and the Browsers will then be closed. If at least one unfired
   // OnBeforeUnload event is found, handle it with a callback to
   // PostBeforeUnloadHandlers, which upon success will recursively call this
@@ skipping 23 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_