Update MimeSniffingResourceHandler tests to use MockResourceLoader.

content/browser/loader/test_resource_handler.cc

Index: content/browser/loader/test_resource_handler.cc
diff --git a/content/browser/loader/test_resource_handler.cc b/content/browser/loader/test_resource_handler.cc
index ea844640cb5d1e31d39e102961ae1a20d7261cbd..86ae04c87e3712c37c0b9037a63e27d9d0ef425e 100644
--- a/content/browser/loader/test_resource_handler.cc
+++ b/content/browser/loader/test_resource_handler.cc
@@ -132,11 +132,14 @@ bool TestResourceHandler::OnWillRead(scoped_refptr<net::IOBuffer>* buf,
 
   ++on_will_read_called_;
 
-  *buf = buffer_;
-  *buf_size = buffer_size_;
-  memset(buffer_->data(), '\0', buffer_size_);
-  if (!on_will_read_result_)
+  if (!on_will_read_result_) {
     canceled_ = true;
+  } else {
+    *buf = buffer_;
+    *buf_size = buffer_size_;
+    memset(buffer_->data(), '\0', buffer_size_);

[mmenke, 2017/01/10 19:20:34]

This change is to meet MockResourceLoader's expectation that these aren't set on
error (Which seems like a good idea to me...But then, I wrote that check)

+  }
+
   return on_will_read_result_;
 }