First patch in making destructors of refcounted objects private.

printing/printed_document.h

 // Copyright (c) 2006-2008 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 PRINTING_PRINTED_DOCUMENT_H_
 #define PRINTING_PRINTED_DOCUMENT_H_
 
 #include <map>
 
 #include "app/gfx/native_widget_types.h"
@@ skipping 21 matching lines @@
 // will have write access. Sensible functions are protected by a lock.
 // Warning: Once a page is loaded, it cannot be replaced. Pages may be discarded
 // under low memory conditions.
 class PrintedDocument : public base::RefCountedThreadSafe<PrintedDocument> {
  public:
   // The cookie shall be unique and has a specific relationship with its
   // originating source and settings.
   PrintedDocument(const PrintSettings& settings,
                   PrintedPagesSource* source,
                   int cookie);
-  ~PrintedDocument();
 
   // Sets a page's data. 0-based. Takes metafile ownership.
   // Note: locks for a short amount of time.
   void SetPage(int page_number, NativeMetafile* metafile, double shrink);
 
   // Retrieves a page. If the page is not available right now, it
   // requests to have this page be rendered and returns false.
   // Note: locks for a short amount of time.
   bool GetPage(int page_number, scoped_refptr<PrintedPage>* page);
 
@@ skipping 46 matching lines @@
   const std::wstring& time() const { return immutable_.time_; }
   const int cookie() const { return immutable_.cookie_; }
 
   // Sets a path where to dump printing output files for debugging. If never set
   // no files are generated.
   static void set_debug_dump_path(const std::wstring& debug_dump_path);
 
   static const std::wstring& debug_dump_path();
 
  private:
+  friend class base::RefCountedThreadSafe<PrintedDocument>;
+
+  ~PrintedDocument();
+
   // Array of data for each print previewed page.
   typedef std::map<int, scoped_refptr<PrintedPage> > PrintedPages;
 
   // Contains all the mutable stuff. All this stuff MUST be accessed with the
   // lock held.
   struct Mutable {
     Mutable(PrintedPagesSource* source);
 
     // Source that generates the PrintedPage's (i.e. a TabContents). It will be
     // set back to NULL if the source is deleted before this object.
@@ skipping 67 matching lines @@
 
   // All the immutable members.
   const Immutable immutable_;
 
   DISALLOW_EVIL_CONSTRUCTORS(PrintedDocument);
 };
 
 }  // namespace printing
 
 #endif  // PRINTING_PRINTED_DOCUMENT_H_