| Index: chrome/browser/download/notification/download_item_notification.cc
|
| diff --git a/chrome/browser/download/notification/download_item_notification.cc b/chrome/browser/download/notification/download_item_notification.cc
|
| index 67c699ef748c49928c9260b75b5c9909469fd086..531d226f271e586b20edaaaba8e7824c176c8553 100644
|
| --- a/chrome/browser/download/notification/download_item_notification.cc
|
| +++ b/chrome/browser/download/notification/download_item_notification.cc
|
| @@ -11,6 +11,7 @@
|
| #include "base/strings/utf_string_conversions.h"
|
| #include "base/task_scheduler/post_task.h"
|
| #include "build/build_config.h"
|
| +#include "chrome/app/vector_icons/vector_icons.h"
|
| #include "chrome/browser/browser_process.h"
|
| #include "chrome/browser/download/download_crx_util.h"
|
| #include "chrome/browser/download/download_item_model.h"
|
| @@ -42,9 +43,9 @@
|
| #include "ui/gfx/color_palette.h"
|
| #include "ui/gfx/image/image.h"
|
| #include "ui/gfx/paint_vector_icon.h"
|
| -#include "ui/gfx/vector_icons_public.h"
|
| #include "ui/message_center/message_center.h"
|
| #include "ui/message_center/message_center_style.h"
|
| +#include "ui/vector_icons/vector_icons.h"
|
|
|
| #if defined(OS_CHROMEOS)
|
| #include "chrome/browser/chromeos/note_taking_helper.h"
|
| @@ -182,7 +183,6 @@ DownloadItemNotification::DownloadItemNotification(
|
| : item_(item),
|
| message_center_(manager->message_center()),
|
| weak_factory_(this) {
|
| -
|
| // Creates the notification instance. |title|, |body| and |icon| will be
|
| // overridden by UpdateNotificationData() below.
|
| notification_.reset(new Notification(
|
| @@ -498,9 +498,9 @@ void DownloadItemNotification::UpdateNotificationIcon() {
|
| ? IDR_DOWNLOAD_NOTIFICATION_WARNING_BAD
|
| : IDR_DOWNLOAD_NOTIFICATION_WARNING_UNWANTED);
|
| #else
|
| - SetNotificationVectorIcon(
|
| - gfx::VectorIconId::WARNING,
|
| - model.MightBeMalicious() ? gfx::kGoogleRed700 : gfx::kGoogleYellow700);
|
| + SetNotificationVectorIcon(ui::kWarningIcon, model.MightBeMalicious()
|
| + ? gfx::kGoogleRed700
|
| + : gfx::kGoogleYellow700);
|
| #endif
|
| return;
|
| }
|
| @@ -514,12 +514,11 @@ void DownloadItemNotification::UpdateNotificationIcon() {
|
| #if defined(OS_MACOSX)
|
| SetNotificationIcon(IDR_DOWNLOAD_NOTIFICATION_INCOGNITO);
|
| #else
|
| - SetNotificationVectorIcon(gfx::VectorIconId::FILE_DOWNLOAD_INCOGNITO,
|
| + SetNotificationVectorIcon(kFileDownloadIncognitoIcon,
|
| gfx::kChromeIconGrey);
|
| #endif
|
| } else {
|
| - SetNotificationVectorIcon(gfx::VectorIconId::FILE_DOWNLOAD,
|
| - gfx::kGoogleBlue500);
|
| + SetNotificationVectorIcon(kFileDownloadIcon, gfx::kGoogleBlue500);
|
| }
|
| break;
|
|
|
| @@ -527,8 +526,7 @@ void DownloadItemNotification::UpdateNotificationIcon() {
|
| #if defined(OS_MACOSX)
|
| SetNotificationIcon(IDR_DOWNLOAD_NOTIFICATION_ERROR);
|
| #else
|
| - SetNotificationVectorIcon(gfx::VectorIconId::ERROR_CIRCLE,
|
| - gfx::kGoogleRed700);
|
| + SetNotificationVectorIcon(ui::kErrorCircleIcon, gfx::kGoogleRed700);
|
| #endif
|
| break;
|
|
|
| @@ -555,20 +553,14 @@ void DownloadItemNotification::OnDownloadRemoved(content::DownloadItem* item) {
|
| }
|
|
|
| void DownloadItemNotification::SetNotificationIcon(int resource_id) {
|
| - if (image_resource_id_ == resource_id)
|
| - return;
|
| ui::ResourceBundle& bundle = ui::ResourceBundle::GetSharedInstance();
|
| - image_resource_id_ = resource_id;
|
| - notification_->set_icon(bundle.GetImageNamed(image_resource_id_));
|
| + notification_->set_icon(bundle.GetImageNamed(resource_id));
|
| }
|
|
|
| -void DownloadItemNotification::SetNotificationVectorIcon(gfx::VectorIconId id,
|
| - SkColor color) {
|
| - if (vector_icon_params_ == std::make_pair(id, color))
|
| - return;
|
| - vector_icon_params_ = std::make_pair(id, color);
|
| - image_resource_id_ = 0;
|
| - notification_->set_icon(gfx::Image(gfx::CreateVectorIcon(id, 40, color)));
|
| +void DownloadItemNotification::SetNotificationVectorIcon(
|
| + const gfx::VectorIcon& icon,
|
| + SkColor color) {
|
| + notification_->set_icon(gfx::Image(gfx::CreateVectorIcon(icon, 40, color)));
|
| }
|
|
|
| void DownloadItemNotification::DisablePopup() {
|
|
|