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

Unified Diff: chrome/browser/chromeos/login/screen_locker.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/chromeos/login/screen_locker.cc
diff --git a/chrome/browser/chromeos/login/screen_locker.cc b/chrome/browser/chromeos/login/screen_locker.cc
index 1331aa5b16eb9c3fddd1856c6e7d8e05af1e0b23..e952542bb8f4288dffdaa935fffdf73fa11c12f2 100644
--- a/chrome/browser/chromeos/login/screen_locker.cc
+++ b/chrome/browser/chromeos/login/screen_locker.cc
@@ -37,9 +37,9 @@
#include "chrome/browser/ui/browser_window.h"
#include "chrome/common/chrome_notification_types.h"
#include "chrome/common/chrome_switches.h"
-#include "content/browser/user_metrics.h"
#include "content/public/browser/browser_thread.h"
#include "content/public/browser/notification_service.h"
+#include "content/public/browser/user_metrics.h"
#include "googleurl/src/gurl.h"
#include "grit/generated_resources.h"
#include "third_party/cros_system_api/window_manager/chromeos_wm_ipc_enums.h"
@@ -54,6 +54,7 @@
#endif
using content::BrowserThread;
+using content::UserMetricsAction;
namespace {
@@ -217,7 +218,7 @@ void ScreenLocker::Init() {
void ScreenLocker::OnLoginFailure(const LoginFailure& error) {
DVLOG(1) << "OnLoginFailure";
- UserMetrics::RecordAction(UserMetricsAction("ScreenLocker_OnLoginFailure"));
+ content::RecordAction(UserMetricsAction("ScreenLocker_OnLoginFailure"));
if (authentication_start_time_.is_null()) {
LOG(ERROR) << "authentication_start_time_ is not set";
} else {
@@ -323,7 +324,7 @@ void ScreenLocker::Signout() {
// TODO(flackr): For proper functionality, check if (error_info) is NULL
// (crbug.com/105267).
delegate_->ClearErrors();
- UserMetrics::RecordAction(UserMetricsAction("ScreenLocker_Signout"));
+ content::RecordAction(UserMetricsAction("ScreenLocker_Signout"));
#if defined(TOOLKIT_USES_GTK)
WmIpc::instance()->NotifyAboutSignout();
#endif
@@ -348,7 +349,7 @@ void ScreenLocker::SetLoginStatusConsumer(
// static
void ScreenLocker::Show() {
DVLOG(1) << "In ScreenLocker::Show";
- UserMetrics::RecordAction(UserMetricsAction("ScreenLocker_Show"));
+ content::RecordAction(UserMetricsAction("ScreenLocker_Show"));
DCHECK(MessageLoop::current()->type() == MessageLoop::TYPE_UI);
// Check whether the currently logged in user is a guest account and if so,

Powered by Google App Engine
This is Rietveld 408576698