Chromium Code Reviews

Unified Diff: chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h

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.
Jump to:
View side-by-side diff with in-line comments
Index: chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h
diff --git a/chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h b/chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h
index f0ea79bac5ba85226b3c5c8351005924fff61b02..43b64184a8c9a21803f35726e6718a15c75c7239 100644
--- a/chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h
+++ b/chrome/browser/notifications/sync_notifier/chrome_notifier_delegate.h
@@ -5,9 +5,11 @@
#ifndef CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_CHROME_NOTIFIER_DELEGATE_H_
#define CHROME_BROWSER_NOTIFICATIONS_SYNC_NOTIFIER_CHROME_NOTIFIER_DELEGATE_H_
+#include <map>
#include <string>
#include "chrome/browser/notifications/notification_delegate.h"
+#include "chrome/browser/notifications/sync_notifier/synced_notification_stats.h"
#include "url/gurl.h"
namespace notifier {
@@ -42,6 +44,9 @@ class ChromeNotifierDelegate : public NotificationDelegate {
const std::string notification_id_;
ChromeNotifierService* const chrome_notifier_;
+ typedef std::map<std::string, SyncedNotificationStats> StatsCollection;
+ StatsCollection stats_;
+
DISALLOW_COPY_AND_ASSIGN(ChromeNotifierDelegate);
};

Powered by Google App Engine