Chromium Code Reviews| Index: chrome/browser/notifications/platform_notification_service_impl.cc |
| diff --git a/chrome/browser/notifications/platform_notification_service_impl.cc b/chrome/browser/notifications/platform_notification_service_impl.cc |
| index 3c125e4f60aa17f64a7ecd68622ea802852966f5..56adbc1d65adae809840f9dade778ccc3465bfff 100644 |
| --- a/chrome/browser/notifications/platform_notification_service_impl.cc |
| +++ b/chrome/browser/notifications/platform_notification_service_impl.cc |
| @@ -486,6 +486,11 @@ Notification PlatformNotificationServiceImpl::CreateNotificationFromData( |
| notification.set_renotify(notification_data.renotify); |
| notification.set_silent(notification_data.silent); |
| + // TODO(peter): Handle different screen densities instead of always using the |
| + // 1x bitmap - crbug.com/585815. |
| + notification.set_small_image( |
| + gfx::Image::CreateFrom1xBitmap(notification_resources.small_icon)); |
|
Peter Beverloo
2016/03/03 17:55:38
What does this actually do on desktop? Do we try t
Michael van Ouwerkerk
2016/03/09 18:28:25
If there are no buttons, they render on top of eac
|
| + |
| // Developer supplied action buttons. |
| std::vector<message_center::ButtonInfo> buttons; |
| for (size_t i = 0; i < notification_data.actions.size(); i++) { |