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

chrome/browser/printing/print_preview_tab_controller.h

 // Copyright (c) 2011 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.
 
 // For print preview, a print preview (PP) tab is linked with the initiator tab
 // that initiated the printing operation. If the tab initiates a second
 // printing operation while the first print preview tab is still open, that PP
 // tab is focused/activated. There may be more than one PP tab open. There is a
 // 1:1 relationship between PP tabs and initiating tabs. This class manages PP
 // tabs and initiator tabs.
@@ skipping 18 matching lines @@
 }
 
 namespace printing {
 
 class PrintPreviewTabController
     : public base::RefCounted<PrintPreviewTabController>,
       public content::NotificationObserver {
  public:
   PrintPreviewTabController();
 
-  virtual ~PrintPreviewTabController();
-
   static PrintPreviewTabController* GetInstance();
 
   // Initiate print preview for |initiator_tab|.
   // Call this instead of GetOrCreatePreviewTab().
   static void PrintPreview(TabContentsWrapper* initiator_tab);
 
   // Get/Create the print preview tab for |initiator_tab|.
   // Exposed for unit tests.
   TabContentsWrapper* GetOrCreatePreviewTab(TabContentsWrapper* initiator_tab);
 
@@ skipping 23 matching lines @@
   bool is_creating_print_preview_tab() const;
 
  private:
   friend class base::RefCounted<PrintPreviewTabController>;
 
   // 1:1 relationship between initiator tab and print preview tab.
   // Key: Preview tab.
   // Value: Initiator tab.
   typedef std::map<TabContentsWrapper*, TabContentsWrapper*> PrintPreviewTabMap;
 
+  virtual ~PrintPreviewTabController();
+
   // Handler for the RENDERER_PROCESS_CLOSED notification. This is observed when
   // the initiator renderer crashed.
   void OnRendererProcessClosed(content::RenderProcessHost* rph);
 
   // Handler for the TAB_CONTENTS_DESTROYED notification. This is observed when
   // either tab is closed.
   void OnTabContentsDestroyed(TabContentsWrapper* tab);
 
   // Handler for the NAV_ENTRY_COMMITTED notification. This is observed when the
   // renderer is navigated to a different page.
@@ skipping 29 matching lines @@
   // Whether the PrintPreviewTabController is in the middle of creating a
   // print preview tab.
   bool is_creating_print_preview_tab_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintPreviewTabController);
 };
 
 }  // namespace printing
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_PREVIEW_TAB_CONTROLLER_H_