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

chrome/browser/status_icons/desktop_notification_balloon.cc

 // Copyright (c) 2012 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.
 
 #include "chrome/browser/status_icons/desktop_notification_balloon.h"
 
 #include "base/bind.h"
 #include "base/string_number_conversions.h"
 #include "base/threading/thread_restrictions.h"
 #include "chrome/browser/browser_process.h"
@@ skipping 14 matching lines @@
 // Prefix added to the notification ids.
 const char kNotificationPrefix[] = "desktop_notification_balloon.";
 
 // Timeout for automatically dismissing the notification balloon.
 const size_t kTimeoutSeconds = 6;
 
 class DummyNotificationDelegate : public NotificationDelegate {
  public:
   explicit DummyNotificationDelegate(const std::string& id)
       : id_(kNotificationPrefix + id) {}
-  virtual ~DummyNotificationDelegate() {}
 
   virtual void Display() OVERRIDE {
     MessageLoop::current()->PostDelayedTask(
         FROM_HERE,
         base::Bind(&CloseBalloon, id()),
         base::TimeDelta::FromSeconds(kTimeoutSeconds));
   }
   virtual void Error() OVERRIDE {}
   virtual void Close(bool by_user) OVERRIDE {}
   virtual void Click() OVERRIDE {}
   virtual std::string id() const OVERRIDE { return id_; }
 
  private:
+  virtual ~DummyNotificationDelegate() {}
+
   std::string id_;
 };
 
 }  // anonymous namespace
 
 int DesktopNotificationBalloon::id_count_ = 1;
 
 DesktopNotificationBalloon::DesktopNotificationBalloon() {
 }
 
@@ skipping 16 matching lines @@
       GURL(), content_url, string16(), string16(),
       new DummyNotificationDelegate(base::IntToString(id_count_++))));
 
   // Allowing IO access is required here to cover the corner case where
   // there is no last used profile and the default one is loaded.
   // IO access won't be required for normal uses.
   base::ThreadRestrictions::ScopedAllowIO allow_io;
   g_browser_process->notification_ui_manager()->Add(
       *notification_.get(), ProfileManager::GetLastUsedProfile());
 }