Update MojoAsyncResourceHandler tests to use MockResourceLoader.

content/browser/loader/mock_resource_loader.h

 // Copyright 2016 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.
 
 #ifndef CONTENT_BROWSER_LOADER_MOCK_RESOURCE_LOADER_H_
 #define CONTENT_BROWSER_LOADER_MOCK_RESOURCE_LOADER_H_
 
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
+#include "base/run_loop.h"
+#include "base/strings/string_piece.h"
 #include "content/browser/loader/resource_controller.h"
+#include "net/base/io_buffer.h"
 #include "net/base/net_errors.h"
 
 class GURL;
 
 namespace net {
 struct RedirectInfo;
 class URLRequestStatus;
 }
 
 namespace content {
@@ skipping 25 matching lines @@
   };
 
   // These all run the corresponding methods on ResourceHandler, along with
   // basic sanity checks for the behavior of the handler. Each check returns the
   // current status of the ResourceLoader.
   Status OnWillStart(const GURL& url);
   Status OnRequestRedirected(const net::RedirectInfo& redirect_info,
                              ResourceResponse* response);
   Status OnResponseStarted(ResourceResponse* response);
   Status OnWillRead(int min_size);
-  Status OnReadCompleted(int bytes_read);
+  Status OnReadCompleted(base::StringPiece bytes);
   Status OnResponseCompleted(const net::URLRequestStatus& status);
 
+  // Simulates an out-of-band cancel from some source other than the
+  // ResourceHandler |this| was created with (like a putative higher layer

[Randy Smith (Not in Mondays), 2016/12/21 16:47:37]

Higher layer?  Maybe this is just a vocabulary issue, but I think of the RH that
the RL points to as being the highest layer RH.

[mmenke, 2017/01/05 18:50:25]

Actually, I was just thinking in terms of the MojoAsyncResourceHandler, where it
would in fact be a ResourceHandler above it (Since none are below it), but the
MockResourceLoader does rather treat the ResourceHandler it's created with as
the top layer ResourceHandler.  Regardless, I've reworded.

+  // ResourceHandler). The difference between this and OnResponseCompleted() is
+  // that this has fewer sanity checks.
+  Status OnResponseCompletedFromExternalOutOfBandCancel(
+      const net::URLRequestStatus& status);
+
+  // Waits until status() is IDLE or CANCELED.  If that's already the case, does
+  // nothing.
+  void WaitUntilIdleOrCanceled();
+
   Status status() const { return status_; }
 
   // Network error passed to the first CancelWithError() / Cancel() call, which
   // is the one the real code uses in the case of multiple cancels.
   int error_code() const { return error_code_; }
 
+  net::IOBuffer* io_buffer() const { return io_buffer_.get(); }
+  int io_buffer_size() const { return io_buffer_size_; }
+
+  void ReleaseIOBuffer();
+
  private:
   // ResourceController implementation.
   void Cancel() override;
   void CancelAndIgnore() override;
   void CancelWithError(int error_code) override;
   void Resume() override;
 
   ResourceHandler* const resource_handler_;
 
   Status status_ = Status::IDLE;
   int error_code_ = net::OK;
 
+  scoped_refptr<net::IOBuffer> io_buffer_;
+  int io_buffer_size_ = 0;
+
+  std::unique_ptr<base::RunLoop> canceled_or_idle_run_loop_;
+
   DISALLOW_COPY_AND_ASSIGN(MockResourceLoader);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_LOADER_TEST_RESOURCE_HANDLER_WRAPPER_H_