Wire up printing on the Mac

chrome/browser/printing/print_job_worker.h

Index: chrome/browser/printing/print_job_worker.h
diff --git a/chrome/browser/printing/print_job_worker.h b/chrome/browser/printing/print_job_worker.h
index fce89122463eba492671c1a15213ae05faf3faed..07b223e87b4496e66304ea89060191d5de72b6a0 100644
--- a/chrome/browser/printing/print_job_worker.h
+++ b/chrome/browser/printing/print_job_worker.h
@@ -5,6 +5,7 @@
 #ifndef CHROME_BROWSER_PRINTING_PRINT_JOB_WORKER_H__
 #define CHROME_BROWSER_PRINTING_PRINT_JOB_WORKER_H__
 
+#include "app/gfx/native_widget_types.h"
 #include "base/task.h"
 #include "base/thread.h"
 #include "printing/page_number.h"
@@ -33,7 +34,7 @@ class PrintJobWorker : public base::Thread {
   // Initializes the print settings. If |ask_user_for_settings| is true, a
   // Print... dialog box will be shown to ask the user his preference.
   void GetSettings(bool ask_user_for_settings,
-                   HWND parent_window,
+                   gfx::NativeWindow parent_window,
                    int document_page_count,
                    bool has_selection);
 
@@ -78,6 +79,16 @@ class PrintJobWorker : public base::Thread {
   // context.
   void OnFailure();
 
+#if defined(OS_MACOSX)
+  // Asks the user for print settings. Must be called on the UI thread.

[pink (ping after 24hrs), 2009/10/13 21:04:51]

Perhaps add that this has to be added because it needs to be called on the main
UI thread, while on windows it can be called on a different thread. Your
existing comment implies that, but it took me some time to understand why win
didn't need this.

[stuartmorgan, 2009/10/13 22:07:43]

Done.

+  void GetSettingsWithUI(gfx::NativeWindow parent_window,
+                         int document_page_count,
+                         bool has_selection);
+#endif
+
+  // Reports settings back to owner_.
+  void GetSettingsDone(PrintingContext::Result result);
+
   // Information about the printer setting.
   PrintingContext printing_context_;