Linux: Refactor printing to be more like Windows/Mac....

chrome/browser/printing/print_job_worker.cc

-// Copyright (c) 2010 The Chromium Authors. All rights reserved.
+// 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.
 
 #include "chrome/browser/printing/print_job_worker.h"
 
 #include "base/message_loop.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/browser_thread.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/common/notification_service.h"
@@ skipping 245 matching lines @@
              &page);
   owner_->message_loop()->PostTask(FROM_HERE, task);
 
   // Preprocess.
   if (printing_context_->NewPage() != PrintingContext::OK) {
     OnFailure();
     return;
   }
 
   // Actual printing.
+#if defined(OS_WIN) || defined(OS_MACOSX)
   document_->RenderPrintedPage(page, printing_context_->context());
+#elif defined(OS_POSIX)
+  document_->RenderPrintedPage(
+      page, reinterpret_cast<PrintingContextCairo*>(printing_context_.get()));

[sanjeevr, 2011/02/17 21:09:48]

Why is this cast done here and not within the implementation of
RenderPrintedPage?

[Lei Zhang, 2011/02/18 00:10:08]

Fixed.

+#endif
 
   // Postprocess.
   if (printing_context_->PageDone() != PrintingContext::OK) {
     OnFailure();
     return;
   }
 
   // Signal everyone that the page is printed.
   task = new NotificationTask();
   task->Init(owner_,
@@ skipping 28 matching lines @@
     printing::PrintJobWorker* obj) {
   DCHECK(!owner_.get());
   owner_ = obj->owner_;
 }
 
 void RunnableMethodTraits<printing::PrintJobWorker>::ReleaseCallee(
     printing::PrintJobWorker* obj) {
   DCHECK_EQ(owner_, obj->owner_);
   owner_ = NULL;
 }