RefCounted types should not have public destructors, chrome/browser/ part 5

chrome/browser/speech/speech_recognition_bubble_controller.h

 // 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.
 
 #ifndef CHROME_BROWSER_SPEECH_SPEECH_RECOGNITION_BUBBLE_CONTROLLER_H_
 #define CHROME_BROWSER_SPEECH_SPEECH_RECOGNITION_BUBBLE_CONTROLLER_H_
 
 #include <map>
 
 #include "base/basictypes.h"
@@ skipping 31 matching lines @@
 
     // Invoked when the user clicks outside the speech recognition info bubble
     // causing it to close and input focus to change.
     virtual void InfoBubbleFocusChanged(int session_id) = 0;
 
    protected:
     virtual ~Delegate() {}
   };
 
   explicit SpeechRecognitionBubbleController(Delegate* delegate);
-  virtual ~SpeechRecognitionBubbleController();
 
   // Creates a new speech recognition UI bubble. One of the SetXxxx methods
   // below need to be called to specify what to display.
   void CreateBubble(int session_id,
                     int render_process_id,
                     int render_view_id,
                     const gfx::Rect& element_rect);
 
   // Indicates to the user that audio hardware is warming up. This also makes
   // the bubble visible if not already visible.
@@ skipping 20 matching lines @@
   virtual void InfoBubbleButtonClicked(
       SpeechRecognitionBubble::Button button) OVERRIDE;
   virtual void InfoBubbleFocusChanged() OVERRIDE;
 
   // content::NotificationObserver implementation.
   virtual void Observe(int type,
                        const content::NotificationSource& source,
                        const content::NotificationDetails& details) OVERRIDE;
 
  private:
+  friend class base::RefCountedThreadSafe<SpeechRecognitionBubbleController>;
+
   // The various calls received by this object and handled in the UI thread.
   enum RequestType {
     REQUEST_SET_WARM_UP_MODE,
     REQUEST_SET_RECORDING_MODE,
     REQUEST_SET_RECOGNIZING_MODE,
     REQUEST_SET_MESSAGE,
     REQUEST_SET_INPUT_VOLUME,
     REQUEST_CLOSE,
   };
 
   enum ManageSubscriptionAction {
     BUBBLE_ADDED,
     BUBBLE_REMOVED
   };
 
+  virtual ~SpeechRecognitionBubbleController();
+
   void InvokeDelegateButtonClicked(int session_id,
                                    SpeechRecognitionBubble::Button button);
   void InvokeDelegateFocusChanged(int session_id);
   void ProcessRequestInUiThread(int session_id,
                                 RequestType type,
                                 const string16& text,
                                 float volume,
                                 float noise_volume);
 
   // Called whenever a bubble was added to or removed from the list. If the
@@ skipping 24 matching lines @@
 // This typedef is to workaround the issue with certain versions of
 // Visual Studio where it gets confused between multiple Delegate
 // classes and gives a C2500 error. (I saw this error on the try bots -
 // the workaround was not needed for my machine).
 typedef SpeechRecognitionBubbleController::Delegate
     SpeechRecognitionBubbleControllerDelegate;
 
 }  // namespace speech
 
 #endif  // CHROME_BROWSER_SPEECH_SPEECH_RECOGNITION_BUBBLE_CONTROLLER_H_