Google Now Message Center Stats

ui/message_center/message_center_types.h

Index: ui/message_center/message_center_types.h
diff --git a/ui/message_center/message_center_types.h b/ui/message_center/message_center_types.h
index 4cfe68e6fad02deb5626e5f3fda6808cbfb2efb0..73aa3e0a8a8d9a587b1d8f5ae872cbd494b5265c 100644
--- a/ui/message_center/message_center_types.h
+++ b/ui/message_center/message_center_types.h
@@ -13,7 +13,15 @@ enum Visibility {
   // When the message center view is being displayed.
   VISIBILITY_MESSAGE_CENTER,
   // When the settings view is being displayed.
-  VISIBILITY_SETTINGS
+  VISIBILITY_SETTINGS,
+};
+
+// Identifies the source of a displayed notification.
+enum NotificationDisplaySource {
+  // Displayed from the message center.
+  MESSAGE_CENTER = 0,

[dewittj, 2014/05/07 18:56:58]

In Chrome the style is to name the constants starting with the enum name.  So
Probably renaming enum to:

enum DisplayType {
  DISPLAY_TYPE_CENTER = 0,
  DISPLAY_TYPE_POPUP
};

would fit better.

[robliao, 2014/05/07 19:58:10]

I'm partial to keeping the source portion since we're talking about source of
the notification.
On 2014/05/07 18:56:58, dewittj wrote:

[dewittj, 2014/05/07 19:59:15]

fine with me, just please adhere to the prefix style.

[robliao, 2014/05/07 20:01:25]

Already done.
On 2014/05/07 19:59:15, dewittj wrote:

+  // Displayed as a popup.
+  POPUP,
 };
 
 }  // namespace message_center