Make WebDataConsumerHandleImpl return OK while zero-length reading of closed pipe.

content/child/web_data_consumer_handle_impl.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 "content/child/web_data_consumer_handle_impl.h"
 
 #include <stdint.h>
 
 #include <limits>
 #include <utility>
@@ skipping 45 matching lines @@
 
   *read_size = 0;
 
   uint32_t size_to_pass = size;
   MojoReadDataFlags flags_to_pass = MOJO_READ_DATA_FLAG_NONE;
   MojoResult rv = mojo::ReadDataRaw(context_->handle().get(), data,
                                     &size_to_pass, flags_to_pass);
   if (rv == MOJO_RESULT_OK)
     *read_size = size_to_pass;
 
+  // Even if there is unread data available, mojo::ReadDataRaw() returns
+  // FAILED_PRECONDITION when |size| is 0 and the producer handle was closed.
+  // But in this case, WebDataConsumerHandle::Reader::read() must return Ok.
+  if (!size && rv == MOJO_RESULT_FAILED_PRECONDITION)
+    return Ok;
+
   return HandleReadResult(rv);
 }
 
 Result WebDataConsumerHandleImpl::ReaderImpl::beginRead(const void** buffer,
                                                         Flags flags,
                                                         size_t* available) {
   // We need this variable definition to avoid a link error.
   const Flags kNone = FlagNone;
   DCHECK_EQ(flags, kNone);
 
@@ skipping 53 matching lines @@
 std::unique_ptr<blink::WebDataConsumerHandle::Reader>
 WebDataConsumerHandleImpl::obtainReader(Client* client) {
   return base::WrapUnique(new ReaderImpl(context_, client));
 }
 
 const char* WebDataConsumerHandleImpl::debugName() const {
   return "WebDataConsumerHandleImpl";
 }
 
 }  // namespace content