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

Unified Diff: ui/message_center/notification_view.h

Issue 11926032: Updated the look of web and basic notifications to match latest mockups. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 11 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: ui/message_center/notification_view.h
diff --git a/ui/message_center/notification_view.h b/ui/message_center/notification_view.h
index 468644c315ffe78792346d1df2f6e06d8a752ade..b8041f9baa398353cd18ecaa0f8ee4e0d0130932 100644
--- a/ui/message_center/notification_view.h
+++ b/ui/message_center/notification_view.h
@@ -10,11 +10,19 @@
namespace message_center {
-// View that displays image and list notifications, and in the future will
-// display all types of notification (web/simple, basic/base, image, and
-// list/multiple-item/inbox/digest).
+// View that displays all current types of notification (web, basic, image, and
+// list). Future notification types may be handled by other classes, in which
+// case instances of those classes would be returned by the
+// ViewForNotification() factory method below.
class NotificationView : public MessageView {
public:
+ // Creates appropriate MessageViews for notifications. Those currently are
+ // always NotificationView instances but in the future may be instances of
+ // other classes, with the class depending on the notification type.
+ static MessageView* ViewForNotification(
+ const NotificationList::Notification& notification,
+ NotificationList::Delegate* list_delegate);
+
NotificationView(NotificationList::Delegate* list_delegate,
const NotificationList::Notification& notification);
virtual ~NotificationView();

Powered by Google App Engine
This is Rietveld 408576698