Printing: Push checking of |context| one level lower, as the different

chrome/browser/printing/print_job_worker.cc

 // Copyright (c) 2006-2008 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/chrome_thread.h"
 #include "chrome/browser/printing/print_job.h"
 #include "chrome/common/notification_service.h"
@@ skipping 119 matching lines @@
       this, &PrintJobWorker::GetSettingsDone, result));
 }
 #endif  // defined(OS_MACOSX) || defined(USE_X11)
 
 void PrintJobWorker::StartPrinting(PrintedDocument* new_document) {
   DCHECK_EQ(message_loop(), MessageLoop::current());
   DCHECK_EQ(page_number_, PageNumber::npos());
   DCHECK_EQ(document_, new_document);
   DCHECK(document_.get());
   DCHECK(new_document->settings().Equals(printing_context_.settings()));
-#if !defined(OS_MACOSX)
-  DCHECK(printing_context_.context());
-#endif
+
   if (!document_.get() || page_number_ != PageNumber::npos() ||
       document_ != new_document) {
     return;
   }
 
   PrintingContext::Result result =
       printing_context_.NewDocument(document_->name());
   if (result != PrintingContext::OK) {
     OnFailure();
     return;
   }
 
   // Try to print already cached data. It may already have been generated for
   // the print preview.
   OnNewPage();
   // Don't touch this anymore since the instance could be destroyed. It happens
   // if all the pages are printed a one sweep and the client doesn't have a
   // handle to us anymore. There's a timing issue involved between the worker
   // thread and the UI thread. Take no chance.
 }
 
 void PrintJobWorker::OnDocumentChanged(PrintedDocument* new_document) {
   DCHECK_EQ(message_loop(), MessageLoop::current());
   DCHECK_EQ(page_number_, PageNumber::npos());
   DCHECK(!new_document ||
          new_document->settings().Equals(printing_context_.settings()));
-#if !defined(OS_MACOSX)
-  DCHECK(printing_context_.context());
-#endif
+
   if (page_number_ != PageNumber::npos())
     return;
 
   document_ = new_document;
 }
 
 void PrintJobWorker::OnNewPage() {
   if (!document_.get()) {
     // Spurious message.
     return;
   }
   // message_loop() could return NULL when the print job is cancelled.
   DCHECK_EQ(message_loop(), MessageLoop::current());
-#if !defined(OS_MACOSX)
-  DCHECK(printing_context_.context());
-  if (!printing_context_.context())
-    return;
-#endif
 
   if (page_number_ == PageNumber::npos()) {
     // Find first page to print.
     int page_count = document_->page_count();
     if (!page_count) {
       // We still don't know how many pages the document contains. We can't
       // start to print the document yet since the header/footer may refer to
       // the document's page count.
       return;
     }
@@ skipping 32 matching lines @@
 }
 
 void PrintJobWorker::DismissDialog() {
   printing_context_.DismissDialog();
 }
 
 void PrintJobWorker::OnDocumentDone() {
   DCHECK_EQ(message_loop(), MessageLoop::current());
   DCHECK_EQ(page_number_, PageNumber::npos());
   DCHECK(document_.get());
-#if !defined(OS_MACOSX)
-  DCHECK(printing_context_.context());
-#endif
 
   if (printing_context_.DocumentDone() != PrintingContext::OK) {
     OnFailure();
     return;
   }
 
   // Tell everyone!
   NotificationTask* task = new NotificationTask();
   task->Init(owner_,
              JobEventDetails::DOC_DONE,
@@ skipping 66 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;
 }