RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/extensions/file_browser_notifications.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/chromeos/extensions/file_browser_notifications.h"
 
 #include "ash/ash_switches.h"
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/message_loop.h"
@@ skipping 86 matching lines @@
 // Simple wrapper class to support ether old chromeos SystemNotifications or
 // new ash notifications.
 class FileBrowserNotifications::NotificationMessage {
  public:
   class Delegate : public NotificationDelegate {
    public:
     Delegate(const base::WeakPtr<FileBrowserNotifications>& host,
              const std::string& id)
         : host_(host),
           id_(id) {}
-    virtual ~Delegate() {}
     virtual void Display() OVERRIDE {}
     virtual void Error() OVERRIDE {}
     virtual void Close(bool by_user) OVERRIDE {
       if (host_)
         host_->HideNotificationById(id_);
     }
     virtual void Click() OVERRIDE {
       // TODO(tbarzic): Show more info page once we have one.
     }
     virtual std::string id() const OVERRIDE { return id_; }
     virtual content::RenderViewHost* GetRenderViewHost() const OVERRIDE {
       return NULL;
     }
 
    private:
+    virtual ~Delegate() {}
+
     base::WeakPtr<FileBrowserNotifications> host_;
     std::string id_;
 
     DISALLOW_COPY_AND_ASSIGN(Delegate);
   };
 
   NotificationMessage(FileBrowserNotifications* host,
                       Profile* profile,
                       NotificationType type,
                       const std::string& notification_id,
@@ skipping 258 matching lines @@
 void FileBrowserNotifications::HideNotificationById(const std::string& id) {
   NotificationMap::iterator it = notification_map_.find(id);
   if (it != notification_map_.end()) {
     delete it->second;
     notification_map_.erase(it);
   } else {
     // Mark as hidden so it does not get shown from a delayed task.
     hidden_notifications_.insert(id);
   }
 }