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

Side by Side Diff: chrome/browser/chromeos/notifications/system_notification.h

Issue 10386206: RefCounted types should not have public destructors, chromeos edition (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase to r143931 Created 8 years, 6 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 | Annotate | Revision Log
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_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_
6 #define CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_ 6 #define CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_
7 #pragma once 7 #pragma once
8 8
9 #include <string> 9 #include <string>
10 10
(...skipping 61 matching lines...) Expand 10 before | Expand all | Expand 10 after
72 class Delegate : public NotificationDelegate { 72 class Delegate : public NotificationDelegate {
73 public: 73 public:
74 explicit Delegate(const std::string& id); 74 explicit Delegate(const std::string& id);
75 virtual void Display() OVERRIDE {} 75 virtual void Display() OVERRIDE {}
76 virtual void Error() OVERRIDE {} 76 virtual void Error() OVERRIDE {}
77 virtual void Close(bool by_user) OVERRIDE {} 77 virtual void Close(bool by_user) OVERRIDE {}
78 virtual void Click() OVERRIDE {} 78 virtual void Click() OVERRIDE {}
79 virtual std::string id() const OVERRIDE; 79 virtual std::string id() const OVERRIDE;
80 virtual content::RenderViewHost* GetRenderViewHost() const OVERRIDE; 80 virtual content::RenderViewHost* GetRenderViewHost() const OVERRIDE;
81 81
82 protected:
83 virtual ~Delegate();
84
82 private: 85 private:
83 std::string id_; 86 std::string id_;
84 87
85 DISALLOW_COPY_AND_ASSIGN(Delegate); 88 DISALLOW_COPY_AND_ASSIGN(Delegate);
86 }; 89 };
87 90
88 void Init(int icon_resource_id); 91 void Init(int icon_resource_id);
89 void ShowNotification(const Notification& notify); 92 void ShowNotification(const Notification& notify);
90 93
91 Profile* profile_; 94 Profile* profile_;
92 BalloonCollectionImplAsh* collection_; 95 BalloonCollectionImplAsh* collection_;
93 scoped_refptr<NotificationDelegate> delegate_; 96 scoped_refptr<NotificationDelegate> delegate_;
94 string16 message_; 97 string16 message_;
95 string16 link_; 98 string16 link_;
96 BalloonViewHost::MessageCallback callback_; 99 BalloonViewHost::MessageCallback callback_;
97 GURL icon_; 100 GURL icon_;
98 string16 title_; 101 string16 title_;
99 bool visible_; 102 bool visible_;
100 bool sticky_; 103 bool sticky_;
101 bool urgent_; 104 bool urgent_;
102 bool show_on_unlock_; 105 bool show_on_unlock_;
103 106
104 DISALLOW_COPY_AND_ASSIGN(SystemNotification); 107 DISALLOW_COPY_AND_ASSIGN(SystemNotification);
105 }; 108 };
106 109
107 } // namespace chromeos 110 } // namespace chromeos
108 111
109 #endif // CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_ 112 #endif // CHROME_BROWSER_CHROMEOS_NOTIFICATIONS_SYSTEM_NOTIFICATION_H_
OLDNEW
« no previous file with comments | « chrome/browser/chromeos/mobile/mobile_activator.cc ('k') | chrome/browser/chromeos/notifications/system_notification.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698