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

Unified Diff: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc

Issue 2771233002: Remove the wrapper functions content::RecordAction et al (Closed)
Patch Set: Rebased Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
diff --git a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
index 1cd8dbb53938649b46e6fe1d8814cff42fc41596..ca0948d47734a9412a59d0b20a80a059e2544530 100644
--- a/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
+++ b/chrome/browser/ui/toolbar/recent_tabs_sub_menu_model.cc
@@ -8,6 +8,7 @@
#include "base/bind.h"
#include "base/metrics/histogram_macros.h"
+#include "base/metrics/user_metrics.h"
#include "base/strings/string_number_conversions.h"
#include "base/strings/utf_string_conversions.h"
#include "build/build_config.h"
@@ -33,7 +34,6 @@
#include "components/strings/grit/components_strings.h"
#include "components/sync_sessions/open_tabs_ui_delegate.h"
#include "components/sync_sessions/synced_session.h"
-#include "content/public/browser/user_metrics.h"
#include "ui/base/accelerators/accelerator.h"
#include "ui/base/l10n/l10n_util.h"
#include "ui/base/resource/resource_bundle.h"
@@ -311,7 +311,7 @@ void RecentTabsSubMenuModel::ExecuteCommand(int command_id, int event_flags) {
if (item.session_tag.empty()) { // Restore tab of local session.
if (service && context) {
- content::RecordAction(
+ base::RecordAction(
base::UserMetricsAction("WrenchMenu_OpenRecentTabFromLocal"));
UMA_HISTOGRAM_ENUMERATION("WrenchMenu.RecentTabsSubMenu",
LOCAL_SESSION_TAB, LIMIT_RECENT_TAB_ACTION);
@@ -326,7 +326,7 @@ void RecentTabsSubMenuModel::ExecuteCommand(int command_id, int event_flags) {
return;
if (tab->navigations.empty())
return;
- content::RecordAction(
+ base::RecordAction(
base::UserMetricsAction("WrenchMenu_OpenRecentTabFromDevice"));
UMA_HISTOGRAM_ENUMERATION("WrenchMenu.RecentTabsSubMenu",
OTHER_DEVICE_TAB, LIMIT_RECENT_TAB_ACTION);
@@ -340,7 +340,7 @@ void RecentTabsSubMenuModel::ExecuteCommand(int command_id, int event_flags) {
int window_items_idx = CommandIdToWindowVectorIndex(command_id);
DCHECK(window_items_idx >= 0 &&
window_items_idx < static_cast<int>(local_window_items_.size()));
- content::RecordAction(
+ base::RecordAction(
base::UserMetricsAction("WrenchMenu_OpenRecentWindow"));
UMA_HISTOGRAM_ENUMERATION("WrenchMenu.RecentTabsSubMenu", RESTORE_WINDOW,
LIMIT_RECENT_TAB_ACTION);
« no previous file with comments | « chrome/browser/ui/toolbar/back_forward_menu_model.cc ('k') | chrome/browser/ui/uma_browsing_activity_observer.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698