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

Unified Diff: chrome/browser/notifications/notification.cc

Issue 580243004: Remove unnecessary image downloading handler of notifications. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: build fix of desktop linux/win Created 6 years, 2 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 | « chrome/browser/notifications/notification.h ('k') | chrome/browser/notifications/notification_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/notifications/notification.cc
diff --git a/chrome/browser/notifications/notification.cc b/chrome/browser/notifications/notification.cc
index 22e79e42b5204ae9289ee9c8b07d0cf880cf9c45..d182c6b58562ffcc9c5988835bf6d91e749c95b3 100644
--- a/chrome/browser/notifications/notification.cc
+++ b/chrome/browser/notifications/notification.cc
@@ -53,9 +53,6 @@ Notification::Notification(const std::string& id,
const Notification& notification)
: message_center::Notification(id, notification),
origin_url_(notification.origin_url()),
- button_one_icon_url_(notification.button_one_icon_url()),
- button_two_icon_url_(notification.button_two_icon_url()),
- image_url_(notification.image_url()),
replace_id_(notification.replace_id()),
delegate_(notification.delegate()) {
}
@@ -63,9 +60,6 @@ Notification::Notification(const std::string& id,
Notification::Notification(const Notification& notification)
: message_center::Notification(notification),
origin_url_(notification.origin_url()),
- button_one_icon_url_(notification.button_one_icon_url()),
- button_two_icon_url_(notification.button_two_icon_url()),
- image_url_(notification.image_url()),
replace_id_(notification.replace_id()),
delegate_(notification.delegate()) {}
@@ -74,9 +68,6 @@ Notification::~Notification() {}
Notification& Notification::operator=(const Notification& notification) {
message_center::Notification::operator=(notification);
origin_url_ = notification.origin_url();
- button_one_icon_url_ = notification.button_one_icon_url();
- button_two_icon_url_ = notification.button_two_icon_url();
- image_url_ = notification.image_url();
replace_id_ = notification.replace_id();
delegate_ = notification.delegate();
return *this;
« no previous file with comments | « chrome/browser/notifications/notification.h ('k') | chrome/browser/notifications/notification_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698