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

Unified Diff: chrome/browser/notifications/notification_test_util.h

Issue 10071036: RefCounted types should not have public destructors, chrome/browser/ part 6 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Implementation fixes Created 8 years, 8 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
Index: chrome/browser/notifications/notification_test_util.h
diff --git a/chrome/browser/notifications/notification_test_util.h b/chrome/browser/notifications/notification_test_util.h
index 9b6dd3480313d7cffd9b624a28b776f7c73d9465..ece73a8660873ea37cb7068a13add6e6c42670eb 100644
--- a/chrome/browser/notifications/notification_test_util.h
+++ b/chrome/browser/notifications/notification_test_util.h
@@ -17,7 +17,6 @@
class MockNotificationDelegate : public NotificationDelegate {
public:
explicit MockNotificationDelegate(const std::string& id);
- virtual ~MockNotificationDelegate();
// NotificationDelegate interface.
virtual void Display() OVERRIDE {}
@@ -27,6 +26,8 @@ class MockNotificationDelegate : public NotificationDelegate {
virtual std::string id() const OVERRIDE;
private:
+ virtual ~MockNotificationDelegate();
+
std::string id_;
DISALLOW_COPY_AND_ASSIGN(MockNotificationDelegate);
« no previous file with comments | « chrome/browser/metrics/field_trial_synchronizer.cc ('k') | chrome/browser/notifications/notification_test_util.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698