Printing: Get settings on the UI thread on all platforms.

chrome/browser/printing/print_job_worker.h

-// Copyright (c) 2006-2008 The Chromium Authors. All rights reserved.
+// 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 CHROME_BROWSER_PRINTING_PRINT_JOB_WORKER_H__
 #define CHROME_BROWSER_PRINTING_PRINT_JOB_WORKER_H__
 #pragma once
 
 #include "base/ref_counted.h"
 #include "base/scoped_ptr.h"
 #include "base/task.h"
@@ skipping 64 matching lines @@
   // Renders a page in the printer.
   void SpoolPage(PrintedPage& page);
 
   // Closes the job since spooling is done.
   void OnDocumentDone();
 
   // Discards the current document, the current page and cancels the printing
   // context.
   void OnFailure();
 
-#if defined(OS_POSIX)
   // Asks the user for print settings. Must be called on the UI thread.
-  // Mac and Linux-only since Windows can display UI from non-main threads.
+  // Required on Mac and Linux. Windows can display UI from non-main threads,
+  // but sticks with this for consistency.
   void GetSettingsWithUI(gfx::NativeView parent_view,
                          int document_page_count,
                          bool has_selection);
 
   // The callback used by PrintingContext::GetSettingsWithUI() to notify this
   // object that the print settings are set.  This is needed in order to bounce
   // back into the IO thread for GetSettingsDone().
   void GetSettingsWithUIDone(PrintingContext::Result result);
-#endif  // defined(OS_POSIX)
 
   // Reports settings back to owner_.
   void GetSettingsDone(PrintingContext::Result result);
 
   // Use the default settings. When using GTK+ or Mac, this can still end up
   // displaying a dialog. So this needs to happen from the UI thread on these
   // systems.
   void UseDefaultSettings();
 
   // Information about the printer setting.
@@ skipping 16 matching lines @@
 
 template <>
 struct RunnableMethodTraits<printing::PrintJobWorker> {
   void RetainCallee(printing::PrintJobWorker* obj);
   void ReleaseCallee(printing::PrintJobWorker* obj);
  private:
   scoped_refptr<printing::PrintJobWorkerOwner> owner_;
 };
 
 #endif  // CHROME_BROWSER_PRINTING_PRINT_JOB_WORKER_H__