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

printing/printed_document_cairo.cc

 // Copyright (c) 2009 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 "printing/printed_document.h"
 
 #include "base/file_util.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "printing/page_number.h"
 #include "printing/printed_page.h"
 
 namespace printing {
 
 void PrintedDocument::RenderPrintedPage(
     const PrintedPage& page, gfx::NativeDrawingContext context) const {
 #ifndef NDEBUG
   {
     // Make sure the page is from our list.
     AutoLock lock(lock_);
     DCHECK(&page == mutable_.pages_.find(page.page_number() - 1)->second.get());
   }
 #endif
 
+  DCHECK(context);

[Marc-Antoine Ruel (Google), 2010/09/25 01:40:13]

There's no implementation, there's no point in checking the value.

[James Hawkins, 2010/09/25 02:27:19]

I'm adding the implementation in a soon-to-be-uploaded CL, and I didn't want to
forget to add the DCHECK to the implementation.

+
   NOTIMPLEMENTED();
 }
 
 }  // namespace printing