Revert 49989 - x

ipc/file_descriptor_set_posix.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 "ipc/file_descriptor_set_posix.h"
 
 #include <sys/types.h>
 #include <sys/stat.h>
 
 #include "base/eintr_wrapper.h"
@@ skipping 12 matching lines @@
   // message should have been transmitted, then closing those with close
   // flags set mirrors the expected behaviour.
   //
   // If this message was received with more descriptors than expected
   // (which could a DOS against the browser by a rogue renderer) then all
   // the descriptors have their close flag set and we free all the extra
   // kernel resources.
   for (unsigned i = consumed_descriptor_highwater_;
        i < descriptors_.size(); ++i) {
     if (descriptors_[i].auto_close)
-      if (HANDLE_EINTR(close(descriptors_[i].fd)) < 0)
-        PLOG(ERROR) << "close";
+      HANDLE_EINTR(close(descriptors_[i].fd));
   }
 }
 
 bool FileDescriptorSet::Add(int fd) {
   if (descriptors_.size() == MAX_DESCRIPTORS_PER_MESSAGE)
     return false;
 
   struct base::FileDescriptor sd;
   sd.fd = fd;
   sd.auto_close = false;
@@ skipping 62 matching lines @@
       return true;
   }
 
   return false;
 }
 
 void FileDescriptorSet::CommitAll() {
   for (std::vector<base::FileDescriptor>::iterator
        i = descriptors_.begin(); i != descriptors_.end(); ++i) {
     if (i->auto_close)
-      if (HANDLE_EINTR(close(i->fd)) < 0)
-        PLOG(ERROR) << "close";
+      HANDLE_EINTR(close(i->fd));
   }
   descriptors_.clear();
   consumed_descriptor_highwater_ = 0;
 }
 
 void FileDescriptorSet::SetDescriptors(const int* buffer, unsigned count) {
   DCHECK_LE(count, MAX_DESCRIPTORS_PER_MESSAGE);
   DCHECK_EQ(descriptors_.size(), 0u);
   DCHECK_EQ(consumed_descriptor_highwater_, 0u);
 
   descriptors_.reserve(count);
   for (unsigned i = 0; i < count; ++i) {
     struct base::FileDescriptor sd;
     sd.fd = buffer[i];
     sd.auto_close = true;
     descriptors_.push_back(sd);
   }
 }