Linux: Simplify printing code and make it more reliable during shutdown.

chrome/browser/printing/print_dialog_gtk.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.
 
 #ifndef CHROME_BROWSER_PRINTING_PRINT_DIALOG_GTK_H_
 #define CHROME_BROWSER_PRINTING_PRINT_DIALOG_GTK_H_
 #pragma once
 
 #include <gtk/gtk.h>
 #include <gtk/gtkprintunixdialog.h>
 
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "content/browser/browser_thread.h"
 #include "printing/print_dialog_gtk_interface.h"
 #include "printing/printing_context_cairo.h"
 #include "ui/base/gtk/gtk_signal.h"
 
-namespace base {
-class WaitableEvent;
-}
-
 namespace printing {
 class Metafile;
 }
 
 using printing::PrintingContextCairo;
 
 // Needs to be freed on the UI thread to clean up its GTK members variables.
 class PrintDialogGtk
     : public printing::PrintDialogGtkInterface,
       public base::RefCountedThreadSafe<PrintDialogGtk,
@@ skipping 17 matching lines @@
  private:
   friend struct BrowserThread::DeleteOnThread<BrowserThread::UI>;
   friend class DeleteTask<PrintDialogGtk>;
 
   explicit PrintDialogGtk(PrintingContextCairo* context);
   virtual ~PrintDialogGtk();
 
   // Handles dialog response.
   CHROMEGTK_CALLBACK_1(PrintDialogGtk, void, OnResponse, int);
 
-  // Saves data in |metafile| to disk for document named |document_name|.
-  void SaveDocumentToDisk(const printing::Metafile* metafile,
-                          const string16& document_name);
-
   // Prints document named |document_name|.
   void SendDocumentToPrinter(const string16& document_name);
 
   // Handles print job response.
   static void OnJobCompletedThunk(GtkPrintJob* print_job,
                                   gpointer user_data,
                                   GError* error);
   void OnJobCompleted(GtkPrintJob* print_job, GError* error);
 
   // Helper function for initializing |context_|'s PrintSettings with a given
   // set of |page_ranges|.
   void InitPrintSettings(const printing::PageRanges& page_ranges);
 
   // Printing dialog callback.
   PrintingContextCairo::PrintSettingsCallback* callback_;
   PrintingContextCairo* context_;
 
   // Print dialog settings. PrintDialogGtk owns |dialog_| and holds references
   // to the other objects.
   GtkWidget* dialog_;
   GtkPrintSettings* gtk_settings_;
   GtkPageSetup* page_setup_;
   GtkPrinter* printer_;
 
-  // Event to signal when save document finishes.
-  scoped_ptr<base::WaitableEvent> save_document_event_;
-
   FilePath path_to_pdf_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintDialogGtk);
 };
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_DIALOG_GTK_H_