Add buffering to MimeSniffingResourceHandler.

content/browser/loader/redirect_to_file_resource_handler_unittest.cc

 // Copyright 2017 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/redirect_to_file_resource_handler.h"
 
 #include <algorithm>
 #include <limits>
 #include <memory>
 #include <string>
@@ skipping 13 matching lines @@
 #include "base/strings/string_piece.h"
 #include "base/threading/thread_task_runner_handle.h"
 #include "content/browser/loader/mock_resource_loader.h"
 #include "content/browser/loader/temporary_file_stream.h"
 #include "content/browser/loader/test_resource_handler.h"
 #include "content/public/common/resource_response.h"
 #include "content/public/test/test_browser_thread_bundle.h"
 #include "net/base/completion_callback.h"
 #include "net/base/file_stream.h"
 #include "net/base/io_buffer.h"
-#include "net/base/mime_sniffer.h"
 #include "net/base/net_errors.h"
 #include "net/base/request_priority.h"
 #include "net/url_request/url_request.h"
 #include "net/url_request/url_request_context.h"
 #include "net/url_request/url_request_status.h"
 #include "net/url_request/url_request_test_util.h"
 #include "storage/browser/blob/shareable_file_reference.h"
 #include "testing/gtest/include/gtest/gtest.h"
 #include "url/gurl.h"
 
 namespace content {
 namespace {
 
 // The maximum size for which the initial read will always be sync, even when
-// the wrote completes asynchronously.  See
-// RedirectToFileResourceHandler::BufIsFull().
+// the write completes asynchronously.
 const int kMaxInitialSyncReadSize =
-    RedirectToFileResourceHandler::kInitialReadBufSize -
-    2 * net::kMaxBytesToSniff - 1;
+    RedirectToFileResourceHandler::kInitialReadBufSize - 1;
 
 // Used to indicate whether FileStream operations and the lower-layer
 // TestResourceHandler operations should complete immediately or by
 // asynchronously invoking a callback. Each test is run with all operations set
 // by default to each mode, though some tests override the mode of some
 // operations.
 enum class CompletionMode {
   SYNC,
   ASYNC,
 };
@@ skipping 762 matching lines @@
   EXPECT_EQ(net::ERR_FAILED, test_handler_->final_status().error());
 }
 
 INSTANTIATE_TEST_CASE_P(/* No prefix needed */,
                         RedirectToFileResourceHandlerTest,
                         testing::Values(CompletionMode::SYNC,
                                         CompletionMode::ASYNC));
 
 }  // namespace
 }  // namespace content