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

chrome/browser/ui/cocoa/first_run_dialog.mm

 // 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.
 
 #import "chrome/browser/ui/cocoa/first_run_dialog.h"
 
 #include "base/bind.h"
 #include "base/mac/bundle_locations.h"
 #include "base/mac/mac_util.h"
 #include "base/memory/ref_counted.h"
@@ skipping 39 matching lines @@
     return NSOrderedDescending;
   else
     return NSOrderedSame;
 }
 
 class FirstRunShowBridge : public base::RefCounted<FirstRunShowBridge> {
  public:
   FirstRunShowBridge(FirstRunDialogController* controller);
 
   void ShowDialog();
+
  private:
+  friend class base::RefCounted<FirstRunShowBridge>;
+
+  ~FirstRunShowBridge();
+
   FirstRunDialogController* controller_;
 };
 
 FirstRunShowBridge::FirstRunShowBridge(
     FirstRunDialogController* controller) : controller_(controller) {
 }
 
 void FirstRunShowBridge::ShowDialog() {
   [controller_ show];
   MessageLoop::current()->QuitNow();
 }
 
+FirstRunShowBridge::~FirstRunShowBridge() {}
+
 // Show the first run UI.
 void ShowFirstRun(Profile* profile) {
 #if defined(GOOGLE_CHROME_BUILD)
   // The purpose of the dialog is to ask the user to enable stats and crash
   // reporting. This setting may be controlled through configuration management
   // in enterprise scenarios. If that is the case, skip the dialog entirely, as
   // it's not worth bothering the user for only the default browser question
   // (which is likely to be forced in enterprise deployments anyway).
   const PrefService::Preference* metrics_reporting_pref =
       g_browser_process->local_state()->FindPreference(
@@ skipping 195 matching lines @@
   [NSApp stopModal];
 }
 
 - (IBAction)learnMore:(id)sender {
   NSString* urlStr = base::SysUTF8ToNSString(chrome::kLearnMoreReportingURL);
   NSURL* learnMoreUrl = [NSURL URLWithString:urlStr];
   [[NSWorkspace sharedWorkspace] openURL:learnMoreUrl];
 }
 
 @end