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

Unified Diff: chrome/browser/tabs/tab_strip_model.cc

Issue 8919017: Split UserMetrics into API vs. implementation. Move API to content/public. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Move to content namespace. Update usages. Update extract_actions tool. Created 9 years 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/tabs/tab_strip_model.cc
diff --git a/chrome/browser/tabs/tab_strip_model.cc b/chrome/browser/tabs/tab_strip_model.cc
index c9731b186a3bd59b87b067b6767609a5a3c6224b..1356740a41941e90d91ae809bd86a2990dac5a68 100644
--- a/chrome/browser/tabs/tab_strip_model.cc
+++ b/chrome/browser/tabs/tab_strip_model.cc
@@ -29,9 +29,11 @@
#include "content/browser/tab_contents/tab_contents.h"
#include "content/browser/tab_contents/tab_contents_delegate.h"
#include "content/browser/tab_contents/tab_contents_view.h"
-#include "content/browser/user_metrics.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/render_process_host.h"
+#include "content/public/browser/user_metrics.h"
+
+using content::UserMetricsAction;
namespace {
@@ -830,12 +832,12 @@ void TabStripModel::ExecuteContextMenuCommand(
DCHECK(command_id > CommandFirst && command_id < CommandLast);
switch (command_id) {
case CommandNewTab:
- UserMetrics::RecordAction(UserMetricsAction("TabContextMenu_NewTab"));
+ content::RecordAction(UserMetricsAction("TabContextMenu_NewTab"));
delegate()->AddBlankTabAt(context_index + 1, true);
break;
case CommandReload: {
- UserMetrics::RecordAction(UserMetricsAction("TabContextMenu_Reload"));
+ content::RecordAction(UserMetricsAction("TabContextMenu_Reload"));
std::vector<int> indices = GetIndicesForCommand(context_index);
for (size_t i = 0; i < indices.size(); ++i) {
TabContentsWrapper* tab = GetTabContentsAt(indices[i]);
@@ -848,7 +850,7 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandDuplicate: {
- UserMetrics::RecordAction(UserMetricsAction("TabContextMenu_Duplicate"));
+ content::RecordAction(UserMetricsAction("TabContextMenu_Duplicate"));
std::vector<int> indices = GetIndicesForCommand(context_index);
// Copy the TabContents off as the indices will change as tabs are
// duplicated.
@@ -864,7 +866,7 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandCloseTab: {
- UserMetrics::RecordAction(UserMetricsAction("TabContextMenu_CloseTab"));
+ content::RecordAction(UserMetricsAction("TabContextMenu_CloseTab"));
std::vector<int> indices = GetIndicesForCommand(context_index);
// Copy the TabContents off as the indices will change as we remove
// things.
@@ -882,7 +884,7 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandCloseOtherTabs: {
- UserMetrics::RecordAction(
+ content::RecordAction(
UserMetricsAction("TabContextMenu_CloseOtherTabs"));
InternalCloseTabs(GetIndicesClosedByCommand(context_index, command_id),
CLOSE_CREATE_HISTORICAL_TAB);
@@ -890,7 +892,7 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandCloseTabsToRight: {
- UserMetrics::RecordAction(
+ content::RecordAction(
UserMetricsAction("TabContextMenu_CloseTabsToRight"));
InternalCloseTabs(GetIndicesClosedByCommand(context_index, command_id),
CLOSE_CREATE_HISTORICAL_TAB);
@@ -898,13 +900,13 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandRestoreTab: {
- UserMetrics::RecordAction(UserMetricsAction("TabContextMenu_RestoreTab"));
+ content::RecordAction(UserMetricsAction("TabContextMenu_RestoreTab"));
delegate_->RestoreTab();
break;
}
case CommandTogglePinned: {
- UserMetrics::RecordAction(
+ content::RecordAction(
UserMetricsAction("TabContextMenu_TogglePinned"));
std::vector<int> indices = GetIndicesForCommand(context_index);
bool pin = WillContextMenuPin(context_index);
@@ -925,7 +927,7 @@ void TabStripModel::ExecuteContextMenuCommand(
}
case CommandBookmarkAllTabs: {
- UserMetrics::RecordAction(
+ content::RecordAction(
UserMetricsAction("TabContextMenu_BookmarkAllTabs"));
delegate_->BookmarkAllTabs();

Powered by Google App Engine
This is Rietveld 408576698