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

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

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 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_ 5 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_
6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_ 6 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
11 #include "base/memory/scoped_ptr.h" 11 #include "base/memory/scoped_ptr.h"
12 #include "base/strings/string16.h" 12 #include "base/strings/string16.h"
13 #include "base/values.h" 13 #include "base/values.h"
14 #include "chrome/browser/notifications/notification_delegate.h" 14 #include "chrome/browser/notifications/notification_delegate.h"
15 #include "third_party/WebKit/public/web/WebTextDirection.h"
16 #include "ui/message_center/notification.h" 15 #include "ui/message_center/notification.h"
17 #include "ui/message_center/notification_types.h" 16 #include "ui/message_center/notification_types.h"
18 #include "url/gurl.h" 17 #include "url/gurl.h"
19 18
20 namespace gfx { 19 namespace gfx {
21 class Image; 20 class Image;
22 } 21 }
23 22
24 // Representation of a notification to be shown to the user. 23 // Representation of a notification to be shown to the user.
25 class Notification : public message_center::Notification { 24 class Notification : public message_center::Notification {
26 public: 25 public:
27 Notification(const GURL& origin_url, 26 Notification(const GURL& origin_url,
28 const base::string16& title, 27 const base::string16& title,
29 const base::string16& body, 28 const base::string16& body,
30 const gfx::Image& icon, 29 const gfx::Image& icon,
31 const base::string16& display_source, 30 const base::string16& display_source,
32 const base::string16& replace_id, 31 const base::string16& replace_id,
33 NotificationDelegate* delegate); 32 NotificationDelegate* delegate);
34 33
35 Notification( 34 Notification(
36 message_center::NotificationType type, 35 message_center::NotificationType type,
37 const GURL& origin_url, 36 const GURL& origin_url,
38 const base::string16& title, 37 const base::string16& title,
39 const base::string16& body, 38 const base::string16& body,
40 const gfx::Image& icon, 39 const gfx::Image& icon,
41 blink::WebTextDirection dir,
42 const message_center::NotifierId& notifier_id, 40 const message_center::NotifierId& notifier_id,
43 const base::string16& display_source, 41 const base::string16& display_source,
44 const base::string16& replace_id, 42 const base::string16& replace_id,
45 const message_center::RichNotificationData& rich_notification_data, 43 const message_center::RichNotificationData& rich_notification_data,
46 NotificationDelegate* delegate); 44 NotificationDelegate* delegate);
47 45
48 Notification(const std::string& id, const Notification& notification); 46 Notification(const std::string& id, const Notification& notification);
49 47
50 Notification(const Notification& notification); 48 Notification(const Notification& notification);
51 ~Notification() override; 49 ~Notification() override;
(...skipping 16 matching lines...) Expand all
68 66
69 // The user-supplied replace ID for the notification. 67 // The user-supplied replace ID for the notification.
70 base::string16 replace_id_; 68 base::string16 replace_id_;
71 69
72 // A proxy object that allows access back to the JavaScript object that 70 // A proxy object that allows access back to the JavaScript object that
73 // represents the notification, for firing events. 71 // represents the notification, for firing events.
74 scoped_refptr<NotificationDelegate> delegate_; 72 scoped_refptr<NotificationDelegate> delegate_;
75 }; 73 };
76 74
77 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_ 75 #endif // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_H_
OLDNEW
« no previous file with comments | « chrome/browser/notifications/message_center_notifications_browsertest.cc ('k') | chrome/browser/notifications/notification.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698