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

Side by Side Diff: chrome/browser/notifications/notification.cc

Issue 873033002: Cleanup: Remove unused blink::WebTextDirection parameter from Notification ctor. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: more fixes Created 5 years, 11 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 12 matching lines...) Expand all
23 origin_url_(origin_url), 23 origin_url_(origin_url),
24 replace_id_(replace_id), 24 replace_id_(replace_id),
25 delegate_(delegate) {} 25 delegate_(delegate) {}
26 26
27 Notification::Notification( 27 Notification::Notification(
28 message_center::NotificationType type, 28 message_center::NotificationType type,
29 const GURL& origin_url, 29 const GURL& origin_url,
30 const base::string16& title, 30 const base::string16& title,
31 const base::string16& body, 31 const base::string16& body,
32 const gfx::Image& icon, 32 const gfx::Image& icon,
33 blink::WebTextDirection dir,
34 const message_center::NotifierId& notifier_id, 33 const message_center::NotifierId& notifier_id,
35 const base::string16& display_source, 34 const base::string16& display_source,
36 const base::string16& replace_id, 35 const base::string16& replace_id,
37 const message_center::RichNotificationData& rich_notification_data, 36 const message_center::RichNotificationData& rich_notification_data,
38 NotificationDelegate* delegate) 37 NotificationDelegate* delegate)
39 : message_center::Notification(type, 38 : message_center::Notification(type,
40 delegate->id(), 39 delegate->id(),
41 title, 40 title,
42 body, 41 body,
43 icon, 42 icon,
(...skipping 21 matching lines...) Expand all
65 64
66 Notification::~Notification() {} 65 Notification::~Notification() {}
67 66
68 Notification& Notification::operator=(const Notification& notification) { 67 Notification& Notification::operator=(const Notification& notification) {
69 message_center::Notification::operator=(notification); 68 message_center::Notification::operator=(notification);
70 origin_url_ = notification.origin_url(); 69 origin_url_ = notification.origin_url();
71 replace_id_ = notification.replace_id(); 70 replace_id_ = notification.replace_id();
72 delegate_ = notification.delegate(); 71 delegate_ = notification.delegate();
73 return *this; 72 return *this;
74 } 73 }
OLDNEW
« no previous file with comments | « chrome/browser/notifications/notification.h ('k') | chrome/browser/notifications/notification_conversion_helper_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698