Move margin processing code to the browser process.

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/compiler_specific.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 printing {
 class Metafile;
+class PrintSettings;
 }
 
 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,
                                         BrowserThread::DeleteOnUIThread> {
  public:
   // Creates and returns a print dialog.
   static printing::PrintDialogGtkInterface* CreatePrintDialog(
       PrintingContextCairo* context);
 
   // printing::PrintDialogGtkInterface implementation.
-  virtual void UseDefaultSettings();
-  virtual bool UpdateSettings(const base::DictionaryValue& settings,
-                              const printing::PageRanges& ranges);
+  virtual void UseDefaultSettings() OVERRIDE;
+  virtual bool UpdateSettings(const base::DictionaryValue& job_settings,
+                              const printing::PageRanges& ranges,
+                              printing::PrintSettings* settings) OVERRIDE;
   virtual void ShowDialog(
-      PrintingContextCairo::PrintSettingsCallback* callback);
+      PrintingContextCairo::PrintSettingsCallback* callback) OVERRIDE;
   virtual void PrintDocument(const printing::Metafile* metafile,
-                             const string16& document_name);
-  virtual void AddRefToDialog();
-  virtual void ReleaseDialog();
+                             const string16& document_name) OVERRIDE;
+  virtual void AddRefToDialog() OVERRIDE;
+  virtual void ReleaseDialog() OVERRIDE;
 
  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);
 
   // 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|.

[Lei Zhang, 2011/10/08 02:02:05]

nit: update comment?

[vandebo (ex-Chrome), 2011/10/15 00:42:50]

Done.

-  void InitPrintSettings(const printing::PageRanges& page_ranges);
+  void InitPrintSettings(const printing::PageRanges& page_ranges,
+                         printing::PrintSettings* settings);
 
   // 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_;
 
   FilePath path_to_pdf_;
 
   DISALLOW_COPY_AND_ASSIGN(PrintDialogGtk);
 };
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_DIALOG_GTK_H_