Revert 49982 - patch

base/message_loop_unittest.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 "base/eintr_wrapper.h"
 #include "base/logging.h"
 #include "base/message_loop.h"
 #include "base/platform_thread.h"
 #include "base/ref_counted.h"
 #include "base/task.h"
@@ skipping 1560 matching lines @@
     base::MessagePumpLibevent::FileDescriptorWatcher controller;
     {
       MessageLoopForIO message_loop;
 
       QuitDelegate delegate;
       message_loop.WatchFileDescriptor(fd,
           true, MessageLoopForIO::WATCH_WRITE, &controller, &delegate);
       // and don't run the message loop, just destroy it.
     }
   }
-  if (HANDLE_EINTR(close(pipefds[0])) < 0)
-    PLOG(WARNING) << "close";
-  if (HANDLE_EINTR(close(pipefds[1])) < 0)
-    PLOG(WARNING) << "close";
+  HANDLE_EINTR(close(pipefds[0]));
+  HANDLE_EINTR(close(pipefds[1]));
 }
 
 TEST(MessageLoopTest, FileDescriptorWatcherDoubleStop) {
   // Verify that it's ok to call StopWatchingFileDescriptor().
   // (Errors only showed up in valgrind.)
   int pipefds[2];
   int err = pipe(pipefds);
   ASSERT_TRUE(err == 0);
   int fd = pipefds[1];
   {
     // Arrange for message loop to live longer than controller.
     MessageLoopForIO message_loop;
     {
       base::MessagePumpLibevent::FileDescriptorWatcher controller;
 
       QuitDelegate delegate;
       message_loop.WatchFileDescriptor(fd,
           true, MessageLoopForIO::WATCH_WRITE, &controller, &delegate);
       controller.StopWatchingFileDescriptor();
     }
   }
-  if (HANDLE_EINTR(close(pipefds[0])) < 0)
-    PLOG(WARNING) << "close";
-  if (HANDLE_EINTR(close(pipefds[1])) < 0)
-    PLOG(WARNING) << "close";
+  HANDLE_EINTR(close(pipefds[0]));
+  HANDLE_EINTR(close(pipefds[1]));
 }
 
 }  // namespace
 
 #endif  // defined(OS_POSIX)