Creates BrowserTabStripTracker to consolidate common code

chrome/browser/extensions/api/tabs/tabs_event_router.h

 // 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
 
 #include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
 #include "base/memory/linked_ptr.h"
 #include "base/scoped_observer.h"
 #include "chrome/browser/extensions/api/tabs/tabs_api.h"
 #include "chrome/browser/ui/browser_list_observer.h"
+#include "chrome/browser/ui/browser_tab_strip_tracker.h"
+#include "chrome/browser/ui/browser_tab_strip_tracker_delegate.h"
 #include "chrome/browser/ui/tabs/tab_strip_model_observer.h"
 #include "components/favicon/core/favicon_driver_observer.h"
 #include "components/ui/zoom/zoom_observer.h"
 #include "content/public/browser/notification_registrar.h"
 #include "extensions/browser/event_router.h"
 
 namespace content {
 class WebContents;
 }
 
 namespace favicon {
 class FaviconDriver;
 }
 
 namespace extensions {
 
 // The TabsEventRouter listens to tab events and routes them to listeners inside
 // extension process renderers.
 // TabsEventRouter will only route events from windows/tabs within a profile to
 // extension processes in the same profile.
 class TabsEventRouter : public TabStripModelObserver,
+                        public BrowserTabStripTrackerDelegate,
                         public chrome::BrowserListObserver,
                         public content::NotificationObserver,
                         public favicon::FaviconDriverObserver,
                         public ui_zoom::ZoomObserver {
  public:
   explicit TabsEventRouter(Profile* profile);
   ~TabsEventRouter() override;
 
+  // BrowserTabStripTrackerDelegate:
+  bool ShouldTrackBrowser(Browser* browser) override;
+
   // chrome::BrowserListObserver

[Peter Kasting, 2015/10/10 00:34:39]

Nit: While here: Colons on these class names?

[sky, 2015/10/12 15:16:21]

Done.

-  void OnBrowserAdded(Browser* browser) override;
-  void OnBrowserRemoved(Browser* browser) override;
   void OnBrowserSetLastActive(Browser* browser) override;
 
   // TabStripModelObserver
   void TabInsertedAt(content::WebContents* contents,
                      int index,
                      bool active) override;
   void TabClosingAt(TabStripModel* tab_strip_model,
                     content::WebContents* contents,
                     int index) override;
   void TabDetachedAt(content::WebContents* contents, int index) override;
@@ skipping 58 matching lines @@
       scoped_ptr<base::ListValue> event_args,
       scoped_ptr<base::ListValue> cross_incognito_args);
 
   // Packages |changed_properties| as a tab updated event for the tab |contents|
   // and dispatches the event to the extension.
   void DispatchTabUpdatedEvent(
       content::WebContents* contents,
       scoped_ptr<base::DictionaryValue> changed_properties);
 
   // Register ourselves to receive the various notifications we are interested
-  // in for a browser.
-  void RegisterForBrowserNotifications(Browser* browser);
-
-  // Register ourselves to receive the various notifications we are interested
   // in for a tab.
   void RegisterForTabNotifications(content::WebContents* contents);
 
   // Removes notifications added in RegisterForTabNotifications.
   void UnregisterForTabNotifications(content::WebContents* contents);
 
   content::NotificationRegistrar registrar_;
 
   // Maintain some information about known tabs, so we can:
   //
@@ skipping 49 matching lines @@
 
   using TabEntryMap = std::map<int, linked_ptr<TabEntry>>;
   TabEntryMap tab_entries_;
 
   // The main profile that owns this event router.
   Profile* profile_;
 
   ScopedObserver<favicon::FaviconDriver, TabsEventRouter>
       favicon_scoped_observer_;
 
+  BrowserTabStripTracker browser_tab_strip_tracker_;
+
   DISALLOW_COPY_AND_ASSIGN(TabsEventRouter);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_