commit bcee2ca34ebcd3b0abc7bc611370d323e55fa62c
author Mike Frysinger <vapier@chromium.org> Wed May 14 16:28:23 2014 -0400
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Thu May 15 23:52:53 2014 +0000
tree b5a991b4af0fcb74173dc50a1026e84e4693d7eb
parent a03a6910f2bad3bcdcc3eb5a8140bb607854f2fe

use IGNORE_EINTR w/close

HANDLE_EINTR is both not safe and not useful on Linux systems.
Switch to IGNORE_EINTR like Chromium has done everywhere.

See http://crbug.com/269623 for details.

BUG=chromium:373154
TEST=`cbuildbot {arm,amd64,x86}-generic-full` passes

Change-Id: Ia2ee7db803366f1305919c4c40c2709e62faae20
Reviewed-on: https://chromium-review.googlesource.com/199823
Reviewed-by: Alex Deymo <deymo@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

file_descriptor.cc

diff --git a/file_descriptor.cc b/file_descriptor.cc
index 255307d..f6b55bb 100644
--- a/file_descriptor.cc
+++ b/file_descriptor.cc
@@ -48,7 +48,7 @@
 
 bool EintrSafeFileDescriptor::Close() {
   CHECK_GE(fd_, 0);
-  if (HANDLE_EINTR(close(fd_)))
+  if (IGNORE_EINTR(close(fd_)))
     return false;
   Reset();
   return true;