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

Unified Diff: chrome/browser/notifications/sync_notifier/synced_notification_stats.cc

Issue 26943002: Add UMA histogram data representing the Synced Notifications feature. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Synced Notifications UMA - remove actions array and fix bug. Created 7 years, 2 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/notifications/sync_notifier/synced_notification_stats.cc
diff --git a/chrome/browser/notifications/sync_notifier/synced_notification_stats.cc b/chrome/browser/notifications/sync_notifier/synced_notification_stats.cc
new file mode 100644
index 0000000000000000000000000000000000000000..6fed8813594b3c07e4478d2280ab501c95b965e5
--- /dev/null
+++ b/chrome/browser/notifications/sync_notifier/synced_notification_stats.cc
@@ -0,0 +1,30 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "chrome/browser/notifications/sync_notifier/synced_notification_stats.h"
+
+#include <string>
+
+#include "base/metrics/histogram.h"
+#include "content/public/browser/user_metrics.h"
+
+namespace notifier {
+
+SyncedNotificationStats::SyncedNotificationStats() {}
+
+SyncedNotificationStats::SyncedNotificationStats(
+ const std::string& id) : id_(id) {}
+
+SyncedNotificationStats::~SyncedNotificationStats() {}
+
+void SyncedNotificationStats::CollectAction(
+ SyncedNotificationActionType type) {
+ DCHECK(!id_.empty());
+
+ UMA_HISTOGRAM_ENUMERATION("SyncedNotifications.Actions",
+ type,
+ SYNCED_NOTIFICATION_ACTION_COUNT);
+}
+
+} // namespace notifier

Powered by Google App Engine
This is Rietveld 408576698