Print Preview: Fix an error in error logging.

chrome/renderer/print_web_view_helper.cc

 // 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/renderer/print_web_view_helper.h"
 
-#if defined(OS_MACOSX) && !defined(USE_SKIA)
-#include <CoreGraphics/CGContext.h>
-#endif
-
 #include <string>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/metrics/histogram.h"
 #include "base/string_number_conversions.h"
 #include "base/utf_string_conversions.h"
 #include "chrome/common/chrome_switches.h"
 #include "chrome/common/print_messages.h"
 #include "chrome/common/render_messages.h"
@@ skipping 21 matching lines @@
 #if defined(OS_POSIX)
 #include "base/process_util.h"
 #include "content/common/view_messages.h"
 #endif
 
 #if defined(USE_SKIA)
 #include "skia/ext/vector_canvas.h"
 #include "skia/ext/vector_platform_device_skia.h"
 #include "third_party/skia/include/core/SkTypeface.h"
 #elif defined(OS_MACOSX)
+#include <CoreGraphics/CGContext.h>
+
 #include "base/mac/scoped_cftyperef.h"
 #include "base/sys_string_conversions.h"
 #include "ui/gfx/scoped_cg_context_save_gstate_mac.h"
 #endif
 
 #if defined(OS_MACOSX)
 using base::mac::ScopedCFTypeRef;
 #endif
 using printing::ConvertPixelsToPoint;
 using printing::ConvertPixelsToPointDouble;
@@ skipping 665 matching lines @@
                                    &(preview_params.metafile_data_handle))) {
     LOG(ERROR) << "CopyMetafileDataToSharedMem failed";
     return false;
   }
   Send(new PrintHostMsg_PagesReadyForPreview(routing_id(), preview_params));
   return true;
 }
 
 void PrintWebViewHelper::OnPrintingDone(bool success) {
   notify_browser_of_print_failure_ = false;
-  if (success == FAIL_PRINT)
+  if (!success)
     LOG(ERROR) << "Failure in OnPrintingDone";
   DidFinishPrinting(success ? OK : FAIL_PRINT);
 }
 
 void PrintWebViewHelper::OnPrintNodeUnderContextMenu() {
   const WebNode& context_menu_node = render_view()->context_menu_node();
   if (context_menu_node.isNull()) {
     NOTREACHED();
     return;
   }
@@ skipping 708 matching lines @@
 const gfx::Size&
 PrintWebViewHelper::PrintPreviewContext::GetPrintCanvasSize() const {
   return prep_frame_view_->GetPrintCanvasSize();
 }
 
 void PrintWebViewHelper::PrintPreviewContext::ClearContext() {
   prep_frame_view_.reset();
   metafile_.reset();
   pages_to_render_.clear();
 }