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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "chrome/browser/notifications/notification.h" 5 #include "chrome/browser/notifications/notification.h"
6 6
7 Notification::Notification(const GURL& origin_url, 7 Notification::Notification(const GURL& origin_url,
8 const base::string16& title, 8 const base::string16& title,
9 const base::string16& body, 9 const base::string16& body,
10 const gfx::Image& icon, 10 const gfx::Image& icon,
(...skipping 35 matching lines...) Expand 10 before | Expand all | Expand 10 after
46 rich_notification_data, 46 rich_notification_data,
47 delegate), 47 delegate),
48 origin_url_(origin_url), 48 origin_url_(origin_url),
49 replace_id_(replace_id), 49 replace_id_(replace_id),
50 delegate_(delegate) {} 50 delegate_(delegate) {}
51 51
52 Notification::Notification(const std::string& id, 52 Notification::Notification(const std::string& id,
53 const Notification& notification) 53 const Notification& notification)
54 : message_center::Notification(id, notification), 54 : message_center::Notification(id, notification),
55 origin_url_(notification.origin_url()), 55 origin_url_(notification.origin_url()),
56 button_one_icon_url_(notification.button_one_icon_url()),
57 button_two_icon_url_(notification.button_two_icon_url()),
58 image_url_(notification.image_url()),
59 replace_id_(notification.replace_id()), 56 replace_id_(notification.replace_id()),
60 delegate_(notification.delegate()) { 57 delegate_(notification.delegate()) {
61 } 58 }
62 59
63 Notification::Notification(const Notification& notification) 60 Notification::Notification(const Notification& notification)
64 : message_center::Notification(notification), 61 : message_center::Notification(notification),
65 origin_url_(notification.origin_url()), 62 origin_url_(notification.origin_url()),
66 button_one_icon_url_(notification.button_one_icon_url()),
67 button_two_icon_url_(notification.button_two_icon_url()),
68 image_url_(notification.image_url()),
69 replace_id_(notification.replace_id()), 63 replace_id_(notification.replace_id()),
70 delegate_(notification.delegate()) {} 64 delegate_(notification.delegate()) {}
71 65
72 Notification::~Notification() {} 66 Notification::~Notification() {}
73 67
74 Notification& Notification::operator=(const Notification& notification) { 68 Notification& Notification::operator=(const Notification& notification) {
75 message_center::Notification::operator=(notification); 69 message_center::Notification::operator=(notification);
76 origin_url_ = notification.origin_url(); 70 origin_url_ = notification.origin_url();
77 button_one_icon_url_ = notification.button_one_icon_url();
78 button_two_icon_url_ = notification.button_two_icon_url();
79 image_url_ = notification.image_url();
80 replace_id_ = notification.replace_id(); 71 replace_id_ = notification.replace_id();
81 delegate_ = notification.delegate(); 72 delegate_ = notification.delegate();
82 return *this; 73 return *this;
83 } 74 }
OLDNEW
« 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