RefCounted types should not have public destructors, chrome/browser/ part 6

chrome/browser/notifications/notification_test_util.h

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_
 #define CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_
 #pragma once
 
 #include <string>
 
 #include "chrome/browser/notifications/notification_object_proxy.h"
 #include "chrome/browser/notifications/balloon.h"
 #include "ui/gfx/size.h"
 
 // NotificationDelegate which does nothing, useful for testing when
 // the notification events are not important.
 class MockNotificationDelegate : public NotificationDelegate {
  public:
   explicit MockNotificationDelegate(const std::string& id);
-  virtual ~MockNotificationDelegate();
 
   // NotificationDelegate interface.
   virtual void Display() OVERRIDE {}
   virtual void Error() OVERRIDE {}
   virtual void Close(bool by_user) OVERRIDE {}
   virtual void Click() OVERRIDE {}
   virtual std::string id() const OVERRIDE;
 
  private:
+  virtual ~MockNotificationDelegate();
+
   std::string id_;
 
   DISALLOW_COPY_AND_ASSIGN(MockNotificationDelegate);
 };
 
 // Mock implementation of Javascript object proxy which logs events that
 // would have been fired on it.  Useful for tests where the sequence of
 // notification events needs to be verified.
 //
 // |Logger| class provided in template must implement method
@@ skipping 45 matching lines @@
   virtual void Close(bool by_user) OVERRIDE;
   virtual gfx::Size GetSize() const OVERRIDE;
   virtual BalloonHost* GetHost() const OVERRIDE;
 
  private:
   // Non-owned pointer.
   Balloon* balloon_;
 };
 
 #endif  // CHROME_BROWSER_NOTIFICATIONS_NOTIFICATION_TEST_UTIL_H_