IPC::ChannelMojo: Don't supress MOJO_RESULT_FAILED_PRECONDITION

ipc/mojo/ipc_message_pipe_reader.cc

 // Copyright 2014 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/mojo/ipc_message_pipe_reader.h"
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 49 matching lines @@
   if (!pipe_wait_id_)
     return;
   mojo::Environment::GetDefaultAsyncWaiter()->CancelWait(pipe_wait_id_);
   pipe_wait_id_ = 0;
 }
 
 void MessagePipeReader::PipeIsReady(MojoResult wait_result) {
   pipe_wait_id_ = 0;
 
   if (wait_result != MOJO_RESULT_OK) {
-    // FAILED_PRECONDITION happens when the pipe is
-    // closed before the waiter is scheduled in a backend thread.
-    if (wait_result != MOJO_RESULT_ABORTED &&
-        wait_result != MOJO_RESULT_FAILED_PRECONDITION) {
-      DLOG(WARNING) << "Pipe got error from the waiter. Closing: "
-                    << wait_result;
+    if (wait_result != MOJO_RESULT_ABORTED) {

[viettrungluu, 2014/08/08 00:36:12]

This is fine, but do you ever expect to see MOJO_RESULT_ABORTED? (If so, when?)

+      // FAILED_PRECONDITION happens every time the peer is dead so
+      // it isn't worth polluting the log message.
+      DLOG_IF(WARNING, wait_result != MOJO_RESULT_FAILED_PRECONDITION)
+          << "Pipe got error from the waiter. Closing: "
+          << wait_result;
       OnPipeError(wait_result);
     }
 
     Close();
     return;
   }
 
   while (pipe_.is_valid()) {
     MojoResult read_result = ReadMessageBytes();
     if (read_result == MOJO_RESULT_SHOULD_WAIT)
@@ skipping 49 matching lines @@
 
 void MessagePipeReader::DelayedDeleter::operator()(
     MessagePipeReader* ptr) const {
   ptr->Close();
   base::MessageLoopProxy::current()->PostTask(
       FROM_HERE, base::Bind(&DeleteNow, ptr));
 }
 
 }  // namespace internal
 }  // namespace IPC