Use mojo for Chrome Loading, Part 1

content/browser/loader/mojo_async_resource_handler_test_util.h

Index: content/browser/loader/mojo_async_resource_handler_test_util.h
diff --git a/content/browser/loader/mojo_async_resource_handler_test_util.h b/content/browser/loader/mojo_async_resource_handler_test_util.h
new file mode 100644
index 0000000000000000000000000000000000000000..4a35fd5b83e16ab889d7130fae085968fccc6766
--- /dev/null
+++ b/content/browser/loader/mojo_async_resource_handler_test_util.h
@@ -0,0 +1,62 @@
+// 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_MOJO_ASYNC_RESOURCE_HANDLER_TEST_UTIL_H_
+#define CONTENT_BROWSER_LOADER_MOJO_ASYNC_RESOURCE_HANDLER_TEST_UTIL_H_
+
+#include "base/callback.h"
+#include "base/macros.h"
+#include "content/common/resource_request_completion_status.h"
+#include "content/common/url_loader.mojom.h"
+#include "content/public/common/resource_response.h"
+#include "mojo/public/c/system/data_pipe.h"
+#include "mojo/public/cpp/bindings/binding.h"
+
+namespace content {
+
+// A FakeURLLoaderClient records URLLoaderClient function calls. It also calls
+// the closure set via set_quit_closure if set, in order to make it possible to
+// create a base::RunLoop, set its quit closure to this client and then run the
+// RunLoop.
+class FakeURLLoaderClient final : public mojom::URLLoaderClient {

[mmenke, 2016/07/25 22:03:53]

The thing is this is a real URLLoaderClient - URLLoaderClient doesn't impose any
behaviors on the client that we're only pretending to do.  So "Fake" seems a
misnomer to me.  More generally, I'd never use the term "Fake" for a class that
implements a pure observer interface, that comes with no contracts in how it
behaves in response to events.

I'd go with "Test", myself (Which is nice and generic and unenlightening,
handily avoiding any such complaints), but I'm perfectly happy with "Stub".

[yhirano, 2016/07/27 16:13:28]

Done.

+ public:
+  FakeURLLoaderClient();
+  ~FakeURLLoaderClient() override;
+
+  void OnReceiveResponse(const ResourceResponseHead& response_head) override;
+  void OnStartLoadingResponseBody(
+      mojo::ScopedDataPipeConsumerHandle body) override;
+  void OnComplete(const ResourceRequestCompletionStatus& status) override;
+
+  bool has_received_response() const { return has_received_response_; }
+  bool has_received_completion() const { return has_received_completion_; }
+  const ResourceResponseHead& response_head() const { return response_head_; }
+  mojo::DataPipeConsumerHandle response_body() { return response_body_.get(); }
+  const ResourceRequestCompletionStatus& completion_status() const {
+    return completion_status_;
+  }
+
+  mojom::URLLoaderClientPtr CreateInterfacePtrAndBind();
+  void Unbind();
+
+  // The set closure will be called when one of On* functions is called.
+  void set_quit_closure(const base::Closure& quit_closure) {
+    quit_closure_ = quit_closure;
+  }
+
+ private:
+  mojo::Binding<mojom::URLLoaderClient> binding_;
+  ResourceResponseHead response_head_;
+  mojo::ScopedDataPipeConsumerHandle response_body_;
+  ResourceRequestCompletionStatus completion_status_;
+  bool has_received_response_ = false;
+  bool has_received_completion_ = false;
+  base::Closure quit_closure_;
+
+  DISALLOW_COPY_AND_ASSIGN(FakeURLLoaderClient);
+};
+
+}  // namespace content
+
+#endif  // CONTENT_BROWSER_LOADER_MOJO_ASYNC_RESOURCE_HANDLER_TEST_UTIL_H_