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

Unified Diff: chrome/browser/ui/ash/system_tray_client.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/ash/system_tray_client.cc
diff --git a/chrome/browser/ui/ash/system_tray_client.cc b/chrome/browser/ui/ash/system_tray_client.cc
index 30465c5676f77a4ebac6c89a6f5657a6e70d43c3..b155cc48203a7dc853c55abea2fbe59d580029ae 100644
--- a/chrome/browser/ui/ash/system_tray_client.cc
+++ b/chrome/browser/ui/ash/system_tray_client.cc
@@ -39,7 +39,6 @@
#include "components/session_manager/core/session_manager.h"
#include "components/user_manager/user_manager.h"
#include "content/public/browser/notification_service.h"
-#include "content/public/browser/user_metrics.h"
#include "content/public/common/service_manager_connection.h"
#include "device/bluetooth/bluetooth_device.h"
#include "extensions/browser/api/vpn_provider/vpn_service.h"
@@ -241,7 +240,7 @@ void SystemTrayClient::ShowBluetoothPairingDialog(
}
void SystemTrayClient::ShowDateSettings() {
- content::RecordAction(base::UserMetricsAction("ShowDateOptions"));
+ base::RecordAction(base::UserMetricsAction("ShowDateOptions"));
// Everybody can change the time zone (even though it is a device setting).
chrome::ShowSettingsSubPageForProfile(ProfileManager::GetActiveUserProfile(),
chrome::kDateTimeSubPage);
@@ -252,12 +251,12 @@ void SystemTrayClient::ShowSetTimeDialog() {
}
void SystemTrayClient::ShowDisplaySettings() {
- content::RecordAction(base::UserMetricsAction("ShowDisplayOptions"));
+ base::RecordAction(base::UserMetricsAction("ShowDisplayOptions"));
ShowSettingsSubPageForActiveUser(chrome::kDisplaySubPage);
}
void SystemTrayClient::ShowPowerSettings() {
- content::RecordAction(base::UserMetricsAction("Tray_ShowPowerOptions"));
+ base::RecordAction(base::UserMetricsAction("Tray_ShowPowerOptions"));
ShowSettingsSubPageForActiveUser(chrome::kPowerSubPage);
}
@@ -268,7 +267,7 @@ void SystemTrayClient::ShowChromeSlow() {
}
void SystemTrayClient::ShowIMESettings() {
- content::RecordAction(base::UserMetricsAction("OpenLanguageOptionsDialog"));
+ base::RecordAction(base::UserMetricsAction("OpenLanguageOptionsDialog"));
ShowSettingsSubPageForActiveUser(chrome::kLanguageOptionsSubPage);
}
@@ -284,7 +283,7 @@ void SystemTrayClient::ShowAccessibilityHelp() {
}
void SystemTrayClient::ShowAccessibilitySettings() {
- content::RecordAction(base::UserMetricsAction("ShowAccessibilitySettings"));
+ base::RecordAction(base::UserMetricsAction("ShowAccessibilitySettings"));
ShowSettingsSubPageForActiveUser(chrome::kAccessibilitySubPage);
}
@@ -296,7 +295,7 @@ void SystemTrayClient::ShowPaletteHelp() {
}
void SystemTrayClient::ShowPaletteSettings() {
- content::RecordAction(base::UserMetricsAction("ShowPaletteOptions"));
+ base::RecordAction(base::UserMetricsAction("ShowPaletteOptions"));
ShowSettingsSubPageForActiveUser(chrome::kStylusSubPage);
}
@@ -354,7 +353,7 @@ void SystemTrayClient::ShowNetworkSettings(const std::string& network_id) {
page = chrome::kNetworkDetailSubPage;
page += "?guid=" + net::EscapeUrlEncodedData(network_id, true);
}
- content::RecordAction(base::UserMetricsAction("OpenInternetOptionsDialog"));
+ base::RecordAction(base::UserMetricsAction("OpenInternetOptionsDialog"));
ShowSettingsSubPageForActiveUser(page);
}
« no previous file with comments | « chrome/browser/ui/ash/chrome_shell_delegate.cc ('k') | chrome/browser/ui/ash/system_tray_delegate_chromeos.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698