Update InterceptingResourceHandler tests to use MockResourceLoader.

content/browser/loader/intercepting_resource_handler.cc

 // 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.
 
 #include "content/browser/loader/intercepting_resource_handler.h"
 
+#include "base/location.h"
 #include "base/logging.h"
 #include "base/strings/string_util.h"
+#include "base/threading/thread_task_runner_handle.h"
 #include "content/public/common/resource_response.h"
 #include "net/base/io_buffer.h"
 #include "net/url_request/url_request.h"
 
 namespace content {
 
 InterceptingResourceHandler::InterceptingResourceHandler(
     std::unique_ptr<ResourceHandler> next_handler,
     net::URLRequest* request)
-    : LayeredResourceHandler(request, std::move(next_handler)) {
+    : LayeredResourceHandler(request, std::move(next_handler)),
+      weak_ptr_factory_(this) {
   next_handler_->SetController(this);
 }
 
 InterceptingResourceHandler::~InterceptingResourceHandler() {}
 
 void InterceptingResourceHandler::SetController(
     ResourceController* controller) {
   if (state_ == State::PASS_THROUGH)
     return LayeredResourceHandler::SetController(controller);
   ResourceHandler::SetController(controller);
@@ skipping 113 matching lines @@
 }
 
 void InterceptingResourceHandler::Resume() {
   DCHECK_NE(State::PASS_THROUGH, state_);
   if (state_ == State::STARTING ||
       state_ == State::WAITING_FOR_ON_READ_COMPLETED) {
     // Uninteresting Resume: just delegate to the original resource controller.
     controller()->Resume();
     return;
   }
-  bool defer = false;
-  if (!DoLoop(&defer)) {
-    controller()->Cancel();
-    return;
-  }
 
-  if (!defer)
-    controller()->Resume();
+  // Can't call DoLoop synchronously, as it may call into |next_handler_|
+  // synchronously, which is what called Resume().
+  base::ThreadTaskRunnerHandle::Get()->PostTask(
+      FROM_HERE, base::Bind(&InterceptingResourceHandler::AdvanceState,
+                            weak_ptr_factory_.GetWeakPtr()));
 }
 
 void InterceptingResourceHandler::UseNewHandler(
     std::unique_ptr<ResourceHandler> new_handler,
     const std::string& payload_for_old_handler) {
   new_handler_ = std::move(new_handler);
   new_handler_->SetController(this);
   payload_for_old_handler_ = payload_for_old_handler;
 }
 
@@ skipping 102 matching lines @@
     if (*defer)
       return true;
   }
 
   state_ = State::PASS_THROUGH;
   first_read_buffer_double_ = nullptr;
   next_handler_->SetController(controller());
   return true;
 }
 
+void InterceptingResourceHandler::AdvanceState() {
+  bool defer = false;
+  if (!DoLoop(&defer)) {
+    controller()->Cancel();
+    return;
+  }
+
+  if (!defer)
+    controller()->Resume();
+}
+
 }  // namespace content