Chromium Code Reviews| Index: ui/message_center/notification_view.cc |
| diff --git a/ui/message_center/notification_view.cc b/ui/message_center/notification_view.cc |
| index 11fd6f191cd4602218ac9044747a696426764a39..5a8133bbb92fda179da5996e4f98adba72fb6b2b 100644 |
| --- a/ui/message_center/notification_view.cc |
| +++ b/ui/message_center/notification_view.cc |
| @@ -158,6 +158,30 @@ void NotificationButton::SetTitle(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."; |
| + } |
|
miket_OOO
2013/01/22 22:39:18
Vertical whitespace after this line would make it
dharcourt
2013/01/23 08:10:12
Done.
|
| + // Currently all roads lead to the generic NotificationView. |
| + return new NotificationView(list_delegate, notification); |
| +} |
| + |
| NotificationView::NotificationView( |
| NotificationList::Delegate* list_delegate, |
| const NotificationList::Notification& notification) |