Minor C++ fixes found by Clang.

printing/pdf_ps_metafile_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/pdf_ps_metafile_cairo.h"
 
 #include <stdio.h>
 
 #include <cairo.h>
 #include <cairo-pdf.h>
@@ skipping 340 matching lines @@
     return false;
   }
 
   bool success = true;
   if (file_util::WriteFileDescriptor(fd.fd, all_pages_.data(),
                                      GetDataSize()) < 0) {
     DLOG(ERROR) << "Failed to save file with fd " << fd.fd;
     success = false;
   }
 
-  if (fd.auto_close)
-    HANDLE_EINTR(close(fd.fd));
+  if (fd.auto_close) {
+    if (HANDLE_EINTR(close(fd.fd)) < 0)

[willchan no longer on Chromium, 2010/03/30 17:13:18]

It's probably a good idea to PLOG here too.

+      success = false;
+  }
+
   return success;
 }
 
 void PdfPsMetafile::CleanUpAll() {
   CleanUpContext(&context_);
   CleanUpSurface(&surface_);
   CleanUpContext(&page_context_);
   CleanUpSurface(&page_surface_);
   current_page_.clear();
   all_pages_.clear();
   skia::VectorPlatformDevice::ClearFontCache();
 }
 
 }  // namespace printing