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

Unified Diff: ui/message_center/notification_view.cc

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
« no previous file with comments | « ui/message_center/notification_view.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/message_center/notification_view.cc
diff --git a/ui/message_center/notification_view.cc b/ui/message_center/notification_view.cc
index d94e008b58f30566e54c85d236f0a959d68e5a2e..85961caa91be876472e5df380b0b85a820b5b4a0 100644
--- a/ui/message_center/notification_view.cc
+++ b/ui/message_center/notification_view.cc
@@ -190,6 +190,31 @@ void NotificationButton::SetTitle(const string16& title) {
namespace message_center {
+// static
+MessageView* NotificationView::ViewForNotification(
+ const NotificationList::Notification& notification,
+ NotificationList::Delegate* list_delegate) {
+ switch (notification.type) {
+ case ui::notifications::NOTIFICATION_TYPE_BASE_FORMAT:
+ case ui::notifications::NOTIFICATION_TYPE_IMAGE:
+ case ui::notifications::NOTIFICATION_TYPE_MULTIPLE:
+ case ui::notifications::NOTIFICATION_TYPE_SIMPLE:
+ break;
+ default:
+ // If the caller asks for an unrecognized kind of view (entirely possible
+ // if an application is running on an older version of this code that
+ // doesn't have the requested kind of notification template), we'll fall
+ // back to a notification instance that will provide at least basic
+ // functionality.
+ LOG(WARNING) << "Unable to fulfill request for unrecognized "
+ << "notification type " << notification.type << ". "
+ << "Falling back to simple notification type.";
+ }
+
+ // Currently all roads lead to the generic NotificationView.
+ return new NotificationView(list_delegate, notification);
+}
+
NotificationView::NotificationView(
NotificationList::Delegate* list_delegate,
const NotificationList::Notification& notification)
« no previous file with comments | « ui/message_center/notification_view.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698