Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(973)

Side by Side Diff: chrome/browser/extensions/api/tabs/tabs_event_router.h

Issue 2335203003: Add metrics to keep track of the tab activate/deactivate cycle (Closed)
Patch Set: Addressed comments Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
6 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_ 6 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 explicit TabsEventRouter(Profile* profile); 47 explicit TabsEventRouter(Profile* profile);
48 ~TabsEventRouter() override; 48 ~TabsEventRouter() override;
49 49
50 // BrowserTabStripTrackerDelegate: 50 // BrowserTabStripTrackerDelegate:
51 bool ShouldTrackBrowser(Browser* browser) override; 51 bool ShouldTrackBrowser(Browser* browser) override;
52 52
53 // chrome::BrowserListObserver: 53 // chrome::BrowserListObserver:
54 void OnBrowserSetLastActive(Browser* browser) override; 54 void OnBrowserSetLastActive(Browser* browser) override;
55 55
56 // TabStripModelObserver: 56 // TabStripModelObserver:
57 void TabInsertedAt(content::WebContents* contents, 57 void TabInsertedAt(TabStripModel* tab_strip_model,
58 content::WebContents* contents,
58 int index, 59 int index,
59 bool active) override; 60 bool active) override;
60 void TabClosingAt(TabStripModel* tab_strip_model, 61 void TabClosingAt(TabStripModel* tab_strip_model,
61 content::WebContents* contents, 62 content::WebContents* contents,
62 int index) override; 63 int index) override;
63 void TabDetachedAt(content::WebContents* contents, int index) override; 64 void TabDetachedAt(content::WebContents* contents, int index) override;
64 void ActiveTabChanged(content::WebContents* old_contents, 65 void ActiveTabChanged(content::WebContents* old_contents,
65 content::WebContents* new_contents, 66 content::WebContents* new_contents,
66 int index, 67 int index,
67 int reason) override; 68 int reason) override;
68 void TabSelectionChanged(TabStripModel* tab_strip_model, 69 void TabSelectionChanged(TabStripModel* tab_strip_model,
69 const ui::ListSelectionModel& old_model) override; 70 const ui::ListSelectionModel& old_model) override;
70 void TabMoved(content::WebContents* contents, 71 void TabMoved(content::WebContents* contents,
71 int from_index, 72 int from_index,
72 int to_index) override; 73 int to_index) override;
73 void TabChangedAt(content::WebContents* contents, 74 void TabChangedAt(content::WebContents* contents,
74 int index, 75 int index,
75 TabChangeType change_type) override; 76 TabChangeType change_type) override;
76 void TabReplacedAt(TabStripModel* tab_strip_model, 77 void TabReplacedAt(TabStripModel* tab_strip_model,
77 content::WebContents* old_contents, 78 content::WebContents* old_contents,
78 content::WebContents* new_contents, 79 content::WebContents* new_contents,
79 int index) override; 80 int index) override;
80 void TabPinnedStateChanged(content::WebContents* contents, 81 void TabPinnedStateChanged(TabStripModel* tab_strip_model,
82 content::WebContents* contents,
81 int index) override; 83 int index) override;
82 84
83 // ZoomObserver: 85 // ZoomObserver:
84 void OnZoomChanged( 86 void OnZoomChanged(
85 const zoom::ZoomController::ZoomChangedEventData& data) override; 87 const zoom::ZoomController::ZoomChangedEventData& data) override;
86 88
87 // favicon::FaviconDriverObserver: 89 // favicon::FaviconDriverObserver:
88 void OnFaviconUpdated(favicon::FaviconDriver* favicon_driver, 90 void OnFaviconUpdated(favicon::FaviconDriver* favicon_driver,
89 NotificationIconType notification_icon_type, 91 NotificationIconType notification_icon_type,
90 const GURL& icon_url, 92 const GURL& icon_url,
(...skipping 116 matching lines...) Expand 10 before | Expand all | Expand 10 after
207 209
208 ScopedObserver<memory::TabManager, TabsEventRouter> 210 ScopedObserver<memory::TabManager, TabsEventRouter>
209 tab_manager_scoped_observer_; 211 tab_manager_scoped_observer_;
210 212
211 DISALLOW_COPY_AND_ASSIGN(TabsEventRouter); 213 DISALLOW_COPY_AND_ASSIGN(TabsEventRouter);
212 }; 214 };
213 215
214 } // namespace extensions 216 } // namespace extensions
215 217
216 #endif // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_ 218 #endif // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698