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

Unified Diff: chrome/browser/download/notification/download_item_notification.h

Issue 2685683004: Move more vector icons to more appropriate directories. (Closed)
Patch Set: fix android? Created 3 years, 10 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: chrome/browser/download/notification/download_item_notification.h
diff --git a/chrome/browser/download/notification/download_item_notification.h b/chrome/browser/download/notification/download_item_notification.h
index 251a1c9ab27f67e23e7587b86c3e166a108a1a6b..53a7d5a4e875c4020960bcdfb4db803dc36b0b50 100644
--- a/chrome/browser/download/notification/download_item_notification.h
+++ b/chrome/browser/download/notification/download_item_notification.h
@@ -26,7 +26,7 @@ class DownloadItemNotificationTest;
}
namespace gfx {
-enum class VectorIconId;
+struct VectorIcon;
}
class DownloadNotificationManagerForProfile;
@@ -70,7 +70,7 @@ class DownloadItemNotification : public DownloadNotification,
// Set icon of the notification.
void SetNotificationIcon(int resource_id);
- void SetNotificationVectorIcon(gfx::VectorIconId id, SkColor color);
+ void SetNotificationVectorIcon(const gfx::VectorIcon& icon, SkColor color);
// Set preview image of the notification. Must be called on IO thread.
void OnImageLoaded(const std::string& image_data);
@@ -114,8 +114,6 @@ class DownloadItemNotification : public DownloadNotification,
// prevents updates after close.
bool closed_ = false;
- int image_resource_id_ = 0;
- std::pair<gfx::VectorIconId, SkColor> vector_icon_params_;
content::DownloadItem::DownloadState previous_download_state_ =
content::DownloadItem::MAX_DOWNLOAD_STATE; // As uninitialized state
bool previous_dangerous_state_ = false;

Powered by Google App Engine
This is Rietveld 408576698