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

Side by Side Diff: chrome/browser/ui/browser_command_controller.cc

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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 #include "chrome/browser/ui/browser_command_controller.h" 5 #include "chrome/browser/ui/browser_command_controller.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 645 matching lines...) Expand 10 before | Expand all | Expand 10 after
656 656
657 void BrowserCommandController::OnSigninAllowedPrefChange() { 657 void BrowserCommandController::OnSigninAllowedPrefChange() {
658 // For unit tests, we don't have a window. 658 // For unit tests, we don't have a window.
659 if (!window()) 659 if (!window())
660 return; 660 return;
661 UpdateShowSyncState(IsShowingMainUI()); 661 UpdateShowSyncState(IsShowingMainUI());
662 } 662 }
663 663
664 // BrowserCommandController, TabStripModelObserver implementation: 664 // BrowserCommandController, TabStripModelObserver implementation:
665 665
666 void BrowserCommandController::TabInsertedAt(WebContents* contents, 666 void BrowserCommandController::TabInsertedAt(TabStripModel* tab_strip_model,
667 WebContents* contents,
667 int index, 668 int index,
668 bool foreground) { 669 bool foreground) {
669 AddInterstitialObservers(contents); 670 AddInterstitialObservers(contents);
670 } 671 }
671 672
672 void BrowserCommandController::TabDetachedAt(WebContents* contents, int index) { 673 void BrowserCommandController::TabDetachedAt(WebContents* contents, int index) {
673 RemoveInterstitialObservers(contents); 674 RemoveInterstitialObservers(contents);
674 } 675 }
675 676
676 void BrowserCommandController::TabReplacedAt(TabStripModel* tab_strip_model, 677 void BrowserCommandController::TabReplacedAt(TabStripModel* tab_strip_model,
(...skipping 495 matching lines...) Expand 10 before | Expand all | Expand 10 after
1172 1173
1173 BrowserWindow* BrowserCommandController::window() { 1174 BrowserWindow* BrowserCommandController::window() {
1174 return browser_->window(); 1175 return browser_->window();
1175 } 1176 }
1176 1177
1177 Profile* BrowserCommandController::profile() { 1178 Profile* BrowserCommandController::profile() {
1178 return browser_->profile(); 1179 return browser_->profile();
1179 } 1180 }
1180 1181
1181 } // namespace chrome 1182 } // namespace chrome
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698