RefCounted types should not have public destructors, chrome/browser/extensions

chrome/browser/extensions/updater/safe_manifest_parser.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/extensions/updater/safe_manifest_parser.h"
 
 #include "base/bind.h"
 #include "base/command_line.h"
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 10 matching lines @@
 
 SafeManifestParser::SafeManifestParser(const std::string& xml,
                                        ManifestFetchData* fetch_data,
                                        const UpdateCallback& update_callback)
     : xml_(xml),
       fetch_data_(fetch_data),
       update_callback_(update_callback) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
 }
 
-SafeManifestParser::~SafeManifestParser() {
-  // If we're using UtilityProcessHost, we may not be destroyed on
-  // the UI or IO thread.
-}
-
 void SafeManifestParser::Start() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   if (!BrowserThread::PostTask(
           BrowserThread::IO, FROM_HERE,
           base::Bind(&SafeManifestParser::ParseInSandbox, this))) {
     NOTREACHED();
   }
 }
 
+SafeManifestParser::~SafeManifestParser() {
+  // If we're using UtilityProcessHost, we may not be destroyed on
+  // the UI or IO thread.
+}
+
 void SafeManifestParser::ParseInSandbox() {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   // TODO(asargent) we shouldn't need to do this branch here - instead
   // UtilityProcessHost should handle it for us. (http://crbug.com/19192)
   bool use_utility_process = content::ResourceDispatcherHost::Get() &&
       !CommandLine::ForCurrentProcess()->HasSwitch(switches::kSingleProcess);
   if (use_utility_process) {
     content::UtilityProcessHost* host = content::UtilityProcessHost::Create(
         this, BrowserThread::UI);
@@ skipping 42 matching lines @@
 
 void SafeManifestParser::OnParseUpdateManifestFailed(
     const std::string& error_message) {
   VLOG(2) << "parsing manifest failed (" << fetch_data_->full_url() << ")";
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   LOG(WARNING) << "Error parsing update manifest:\n" << error_message;
   update_callback_.Run(*fetch_data_, NULL);
 }
 
 }  // namespace extensions