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

chrome/browser/ui/fullscreen_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_UI_FULLSCREEN_CONTROLLER_H_
 #define CHROME_BROWSER_UI_FULLSCREEN_CONTROLLER_H_
 #pragma once
 
 #include "base/basictypes.h"
 #include "base/memory/ref_counted.h"
@@ skipping 19 matching lines @@
 // whereas browser fullscreen mode doesn't. Therefore if a user forces an exit
 // from tab fullscreen, we need to notify the tab so it can stop rendering in
 // its fullscreen mode.
 
 // This class implements fullscreen and mouselock behaviour.
 class FullscreenController : public base::RefCounted<FullscreenController> {
  public:
   FullscreenController(BrowserWindow* window,
                        Profile* profile,
                        Browser* browser);
-  virtual ~FullscreenController();
 
   // Querying.
 
   // Returns true if the window is currently fullscreen and was initially
   // transitioned to fullscreen by a browser (vs tab) mode transition.
   bool IsFullscreenForBrowser() const;
 
   // Returns true if fullscreen has been caused by a tab.
   // The window may still be transitioning, and window_->IsFullscreen()
   // may still return false.
@@ skipping 23 matching lines @@
   void OnTabDeactivated(TabContentsWrapper* contents);
   void OnAcceptFullscreenPermission(const GURL& url,
                                     FullscreenExitBubbleType bubble_type);
   void OnDenyFullscreenPermission(FullscreenExitBubbleType bubble_type);
   void WindowFullscreenStateChanged();
   bool HandleUserPressedEscape();
 
   FullscreenExitBubbleType GetFullscreenExitBubbleType() const;
 
  private:
+  friend class base::RefCounted<FullscreenController>;
+
   enum MouseLockState {
     MOUSELOCK_NOT_REQUESTED,
     // The page requests to lock the mouse and the user hasn't responded to the
     // request.
     MOUSELOCK_REQUESTED,
     // Mouse lock has been allowed by the user.
     MOUSELOCK_ACCEPTED
   };
 
+  virtual ~FullscreenController();
+
   // Notifies the tab that it has been forced out of fullscreen mode if
   // necessary.
   void NotifyTabOfFullscreenExitIfNecessary();
   // Make the current tab exit fullscreen mode if it is in it.
   void ExitTabbedFullscreenModeIfNecessary();
   void UpdateFullscreenExitBubbleContent();
   void NotifyFullscreenChange();
   ContentSetting GetFullscreenSetting(const GURL& url) const;
   ContentSetting GetMouseLockSetting(const GURL& url) const;
 
@@ skipping 19 matching lines @@
   // True if tab fullscreen has been allowed, either by settings or by user
   // clicking the allow button on the fullscreen infobar.
   bool tab_fullscreen_accepted_;
 
   MouseLockState mouse_lock_state_;
 
   DISALLOW_COPY_AND_ASSIGN(FullscreenController);
 };
 
 #endif  // CHROME_BROWSER_UI_FULLSCREEN_CONTROLLER_H_