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

Cuts ~2MB off our .a files (Debug, Linux). Also added the "virtual" keyword on
a whole bunch of virtual dtors that were missing it.

BUG=none
TEST=compiles

Review URL: http://codereview.chromium.org/3522004

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@61100 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 3 insertions, 1 deletions

diff --git a/printing/printed_document.h b/printing/printed_document.h
index e4a3280..dfab7a5 100644
--- a/printing/printed_document.h
+++ b/printing/printed_document.h
@@ -111,7 +111,7 @@ class PrintedDocument : public base::RefCountedThreadSafe<PrintedDocument> {
  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;
@@ -120,6 +120,7 @@ class PrintedDocument : public base::RefCountedThreadSafe<PrintedDocument> {
   // 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.
@@ -146,6 +147,7 @@ class PrintedDocument : public base::RefCountedThreadSafe<PrintedDocument> {
   struct Immutable {
     Immutable(const PrintSettings& settings, PrintedPagesSource* source,
               int cookie);
+    ~Immutable();
 
     // Print settings used to generate this document. Immutable.
     PrintSettings settings_;