RefCounted types should not have public destructors, chromeos edition

chrome/browser/ui/webui/chromeos/proxy_settings_ui.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/ui/webui/chromeos/proxy_settings_ui.h"
 
 #include "base/memory/ref_counted_memory.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/message_loop.h"
 #include "base/values.h"
@@ skipping 18 matching lines @@
 namespace {
 
 class ProxySettingsHTMLSource : public ChromeURLDataManager::DataSource {
  public:
   explicit ProxySettingsHTMLSource(DictionaryValue* localized_strings);
 
   // Called when the network layer has requested a resource underneath
   // the path we registered.
   virtual void StartDataRequest(const std::string& path,
                                 bool is_incognito,
-                                int request_id);
-  virtual std::string GetMimeType(const std::string&) const {
+                                int request_id) OVERRIDE;
+  virtual std::string GetMimeType(const std::string&) const OVERRIDE {
     return "text/html";
   }
 
+ protected:
+  virtual ~ProxySettingsHTMLSource() {}
+
  private:
   scoped_ptr<DictionaryValue> localized_strings_;
 
   DISALLOW_COPY_AND_ASSIGN(ProxySettingsHTMLSource);
 };
 
 ProxySettingsHTMLSource::ProxySettingsHTMLSource(
     DictionaryValue* localized_strings)
     : DataSource(chrome::kChromeUIProxySettingsHost, MessageLoop::current()),
       localized_strings_(localized_strings) {
@@ skipping 50 matching lines @@
   core_handler_->InitializePage();
   proxy_handler_->InitializePage();
   Profile* profile = Profile::FromWebUI(web_ui());
   PrefProxyConfigTracker* proxy_tracker = profile->GetProxyConfigTracker();
   proxy_tracker->UIMakeActiveNetworkCurrent();
   std::string network_name;
   proxy_tracker->UIGetCurrentNetworkName(&network_name);
 }
 
 }  // namespace chromeos