RefCounted types should not have public destructors, chromeos edition

chrome/browser/chromeos/login/help_app_launcher.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/login/help_app_launcher.h"
 
 #include <string>
 
 #include "base/stringprintf.h"
 #include "base/utf_string_conversions.h"
@@ skipping 14 matching lines @@
 
 namespace chromeos {
 
 ///////////////////////////////////////////////////////////////////////////////
 // HelpApp, public:
 
 HelpAppLauncher::HelpAppLauncher(gfx::NativeWindow parent_window)
     : parent_window_(parent_window) {
 }
 
-HelpAppLauncher::~HelpAppLauncher() {}
-
 void HelpAppLauncher::ShowHelpTopic(HelpTopic help_topic_id) {
   Profile* profile = ProfileManager::GetDefaultProfile();
   ExtensionService* service = profile->GetExtensionService();
 
   DCHECK(service);
   if (!service)
     return;
 
   GURL url(base::StringPrintf(kHelpAppFormat,
                               static_cast<int>(help_topic_id)));
   // HelpApp component extension presents only in official builds so we can
   // show help only when the extensions is installed.
   if (service->extensions()->GetByID(url.host()))
     ShowHelpTopicDialog(GURL(url));
 }
 
 ///////////////////////////////////////////////////////////////////////////////
+// HelpApp, protected:
+
+HelpAppLauncher::~HelpAppLauncher() {}
+
+///////////////////////////////////////////////////////////////////////////////
 // HelpApp, private:
 
 void HelpAppLauncher::ShowHelpTopicDialog(const GURL& topic_url) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::UI));
   dialog_.reset(new LoginWebDialog(
       this,
       parent_window_,
       l10n_util::GetStringUTF16(IDS_LOGIN_OOBE_HELP_DIALOG_TITLE),
       topic_url,
       LoginWebDialog::STYLE_BUBBLE));
   dialog_->Show();
 }
 
 }  // namespace chromeos