FBTF: Move ctors/dtors into implementation files. Adds ctors/dtors to non-POD structs.

printing/printed_document.h

 // Copyright (c) 2010 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 "base/lock.h"
@@ skipping 93 matching lines @@
 
   // 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 FilePath& debug_dump_path);
 
   static const FilePath& debug_dump_path();
 
  private:
   friend class base::RefCountedThreadSafe<PrintedDocument>;
 
-  ~PrintedDocument();
+  virtual ~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 {
     explicit Mutable(PrintedPagesSource* source);
+    ~Mutable();
 
     // 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.
     PrintedPagesSource* source_;
 
     // Contains the pages' representation. This is a collection of PrintedPage.
     // Warning: Lock must be held when accessing this member.
     PrintedPages pages_;
 
     // Number of expected pages to be rendered.
     // Warning: Lock must be held when accessing this member.
     int expected_page_count_;
 
     // The total number of pages in the document.
     int page_count_;
 
     // Shrink done in comparison to desired_dpi.
     double shrink_factor;
   };
 
   // Contains all the immutable stuff. All this stuff can be accessed without
   // any lock held. This is because it can't be changed after the object's
   // construction.
   struct Immutable {
     Immutable(const PrintSettings& settings, PrintedPagesSource* source,
               int cookie);
+    ~Immutable();
 
     // Print settings used to generate this document. Immutable.
     PrintSettings settings_;
 
     // Native thread for the render source.
     MessageLoop* source_message_loop_;
 
     // Document name. Immutable.
     string16 name_;
 
@@ skipping 34 matching lines @@
 
   // All the immutable members.
   const Immutable immutable_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintedDocument);
 };
 
 }  // namespace printing
 
 #endif  // PRINTING_PRINTED_DOCUMENT_H_