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

Unified Diff: chrome/browser/notifications/message_center_stats_collector.h

Issue 23902057: Stats collector for message center. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Histograms.xml nit Created 7 years, 3 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/message_center_stats_collector.h
diff --git a/chrome/browser/notifications/message_center_stats_collector.h b/chrome/browser/notifications/message_center_stats_collector.h
new file mode 100644
index 0000000000000000000000000000000000000000..f29a08c08b56479a10bfbb3bf4fe6bf5223ee6a8
--- /dev/null
+++ b/chrome/browser/notifications/message_center_stats_collector.h
@@ -0,0 +1,90 @@
+// 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.
+
+#ifndef CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_STATS_COLLECTOR_H_
+#define CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_STATS_COLLECTOR_H_
+
+#include <set>
+#include <string>
+
+#include "ui/message_center/message_center.h"
+#include "ui/message_center/message_center_observer.h"
+#include "ui/message_center/message_center_types.h"
+
+namespace message_center {
+class MessageCenter;
+}
+
+// MessageCenterStatsCollector sends both raw and per-notification statistics
+// to the UMA servers, if the user has opted in. It observes the message center
+// to gather its data.
+class MessageCenterStatsCollector
+ : public message_center::MessageCenterObserver {
+ public:
+ enum NotificationActionType {
+ NOTIFICATION_ACTION_UNKNOWN,
+ NOTIFICATION_ACTION_ADD,
+ NOTIFICATION_ACTION_UPDATE,
+ NOTIFICATION_ACTION_CLICK,
+ NOTIFICATION_ACTION_BUTTON_CLICK,
+ NOTIFICATION_ACTION_DISPLAY,
+ NOTIFICATION_ACTION_CLOSE_BY_USER,
+ NOTIFICATION_ACTION_CLOSE_BY_SYSTEM,
+ // NOTE: Add new action types only immediately above this line. Also,
+ // make sure the enum list in tools/histogram/histograms.xml is
+ // updated with any change in here.
+ NOTIFICATION_ACTION_COUNT
+ };
+
+ explicit MessageCenterStatsCollector(
+ message_center::MessageCenter* message_center);
+ virtual ~MessageCenterStatsCollector();
+
+ private:
+ // Represents the aggregate stats for each notification.
+ class NotificationStats {
+ public:
+ // Default constructor for map.
+ NotificationStats();
+
+ explicit NotificationStats(const std::string& id);
+ virtual ~NotificationStats();
+
+ // Called when we get an action from the message center.
+ void CollectAction(NotificationActionType type);
+
+ // Sends aggregate data to UMA.
+ void RecordAggregateStats();
+
+ private:
+ std::string id_;
+ bool actions_[NOTIFICATION_ACTION_COUNT];
+ };
+
+ // MessageCenterObserver
+ virtual void OnNotificationAdded(const std::string& notification_id) OVERRIDE;
+ virtual void OnNotificationRemoved(const std::string& notification_id,
+ bool by_user) OVERRIDE;
+ virtual void OnNotificationUpdated(
+ const std::string& notification_id) OVERRIDE;
+ virtual void OnNotificationClicked(
+ const std::string& notification_id) OVERRIDE;
+ virtual void OnNotificationButtonClicked(const std::string& notification_id,
+ int button_index) OVERRIDE;
+ virtual void OnNotificationDisplayed(
+ const std::string& notification_id) OVERRIDE;
+ virtual void OnCenterVisibilityChanged(
+ message_center::Visibility visibility) OVERRIDE;
+ virtual void OnQuietModeChanged(bool in_quiet_mode) OVERRIDE;
+
+ // Weak, global.
+ message_center::MessageCenter* message_center_;
+
+ typedef std::map<std::string,NotificationStats> StatsCollection;
+ StatsCollection stats_;
+
+ DISALLOW_COPY_AND_ASSIGN(MessageCenterStatsCollector);
+};
+
+#endif // CHROME_BROWSER_NOTIFICATIONS_MESSAGE_CENTER_STATS_COLLECTOR_H_

Powered by Google App Engine
This is Rietveld 408576698