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

Unified Diff: ui/message_center/views/message_center_view.h

Issue 75133006: Move knowledge about MessageCenter out of MessageView. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: remover types Created 7 years, 1 month 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: ui/message_center/views/message_center_view.h
diff --git a/ui/message_center/views/message_center_view.h b/ui/message_center/views/message_center_view.h
index e230c382bf8796045622d9bf31be77bec7ebefcd..d1b3bfef96fa3343a528c9ea04827cc65864901f 100644
--- a/ui/message_center/views/message_center_view.h
+++ b/ui/message_center/views/message_center_view.h
@@ -31,6 +31,7 @@ class MessageCenterTray;
class MessageCenterView;
class MessageView;
class MessageListView;
+class NotificationView;
class NotifierSettingsView;
// MessageCenterView ///////////////////////////////////////////////////////////
@@ -87,7 +88,10 @@ class MESSAGE_CENTER_EXPORT MessageCenterView : public views::View,
MessageCenter* message_center_; // Weak reference.
MessageCenterTray* tray_; // Weak reference.
- std::vector<MessageView*> message_views_; // Weak references.
+ // Set of NotificationView objects, each corresponding to a single
+ // notification.
+ typedef std::map<std::string, NotificationView*> NotificationViewsMap;
+ NotificationViewsMap notification_views_; // Weak.
dewittj 2013/11/22 00:33:55 The comment here is confusing since notification_v
Dmitry Titov 2013/11/22 01:13:51 Done.
// Child views.
views::ScrollView* scroller_;
« no previous file with comments | « no previous file | ui/message_center/views/message_center_view.cc » ('j') | ui/message_center/views/message_center_view.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698