Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(259)

Side by Side Diff: content/browser/loader/mojo_async_resource_handler_unittest.cc

Issue 2646343007: [Mojo-Loading] Implement URLLoader::SetPriority (Closed)
Patch Set: rebase Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The Chromium Authors. All rights reserved. 1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "content/browser/loader/mojo_async_resource_handler.h" 5 #include "content/browser/loader/mojo_async_resource_handler.h"
6 6
7 #include <string.h> 7 #include <string.h>
8 8
9 #include <utility> 9 #include <utility>
10 #include <vector> 10 #include <vector>
11 11
12 #include "base/bind.h" 12 #include "base/bind.h"
13 #include "base/callback.h" 13 #include "base/callback.h"
14 #include "base/files/file_path.h" 14 #include "base/files/file_path.h"
15 #include "base/logging.h" 15 #include "base/logging.h"
16 #include "base/macros.h" 16 #include "base/macros.h"
17 #include "base/memory/ptr_util.h" 17 #include "base/memory/ptr_util.h"
18 #include "base/memory/weak_ptr.h" 18 #include "base/memory/weak_ptr.h"
19 #include "base/run_loop.h" 19 #include "base/run_loop.h"
20 #include "base/test/test_simple_task_runner.h" 20 #include "base/test/test_simple_task_runner.h"
21 #include "content/browser/loader/mock_resource_loader.h" 21 #include "content/browser/loader/mock_resource_loader.h"
22 #include "content/browser/loader/resource_controller.h" 22 #include "content/browser/loader/resource_controller.h"
23 #include "content/browser/loader/resource_dispatcher_host_impl.h" 23 #include "content/browser/loader/resource_dispatcher_host_impl.h"
24 #include "content/browser/loader/resource_request_info_impl.h" 24 #include "content/browser/loader/resource_request_info_impl.h"
25 #include "content/browser/loader/resource_scheduler.h"
25 #include "content/browser/loader/test_url_loader_client.h" 26 #include "content/browser/loader/test_url_loader_client.h"
26 #include "content/common/resource_request_completion_status.h" 27 #include "content/common/resource_request_completion_status.h"
27 #include "content/common/url_loader.mojom.h" 28 #include "content/common/url_loader.mojom.h"
28 #include "content/public/browser/appcache_service.h" 29 #include "content/public/browser/appcache_service.h"
29 #include "content/public/browser/navigation_data.h" 30 #include "content/public/browser/navigation_data.h"
30 #include "content/public/browser/resource_context.h" 31 #include "content/public/browser/resource_context.h"
31 #include "content/public/browser/resource_dispatcher_host_delegate.h" 32 #include "content/public/browser/resource_dispatcher_host_delegate.h"
32 #include "content/public/browser/resource_throttle.h" 33 #include "content/public/browser/resource_throttle.h"
33 #include "content/public/browser/stream_info.h" 34 #include "content/public/browser/stream_info.h"
34 #include "content/public/common/previews_state.h" 35 #include "content/public/common/previews_state.h"
35 #include "content/public/common/resource_response.h" 36 #include "content/public/common/resource_response.h"
36 #include "content/public/common/resource_type.h" 37 #include "content/public/common/resource_type.h"
37 #include "content/public/test/test_browser_context.h" 38 #include "content/public/test/test_browser_context.h"
38 #include "content/public/test/test_browser_thread_bundle.h" 39 #include "content/public/test/test_browser_thread_bundle.h"
39 #include "mojo/public/c/system/data_pipe.h" 40 #include "mojo/public/c/system/data_pipe.h"
40 #include "mojo/public/c/system/types.h" 41 #include "mojo/public/c/system/types.h"
41 #include "mojo/public/cpp/bindings/strong_binding.h" 42 #include "mojo/public/cpp/bindings/strong_binding.h"
42 #include "mojo/public/cpp/system/data_pipe.h" 43 #include "mojo/public/cpp/system/data_pipe.h"
43 #include "net/base/auth.h" 44 #include "net/base/auth.h"
44 #include "net/base/net_errors.h" 45 #include "net/base/net_errors.h"
46 #include "net/base/request_priority.h"
45 #include "net/http/http_response_headers.h" 47 #include "net/http/http_response_headers.h"
46 #include "net/http/http_response_info.h" 48 #include "net/http/http_response_info.h"
47 #include "net/http/http_status_code.h" 49 #include "net/http/http_status_code.h"
48 #include "net/http/http_util.h" 50 #include "net/http/http_util.h"
49 #include "net/ssl/client_cert_store.h" 51 #include "net/ssl/client_cert_store.h"
50 #include "net/test/url_request/url_request_mock_data_job.h" 52 #include "net/test/url_request/url_request_mock_data_job.h"
51 #include "net/url_request/url_request.h" 53 #include "net/url_request/url_request.h"
52 #include "net/url_request/url_request_context.h" 54 #include "net/url_request/url_request_context.h"
53 #include "net/url_request/url_request_status.h" 55 #include "net/url_request/url_request_status.h"
54 #include "net/url_request/url_request_test_util.h" 56 #include "net/url_request/url_request_test_util.h"
(...skipping 883 matching lines...) Expand 10 before | Expand all | Expand 10 after
938 940
939 // Expect a upload progress report for the upload completion. 941 // Expect a upload progress report for the upload completion.
940 url_loader_client_.reset_has_received_upload_progress(); 942 url_loader_client_.reset_has_received_upload_progress();
941 set_upload_progress(net::UploadProgress(1000, 1000)); 943 set_upload_progress(net::UploadProgress(1000, 1000));
942 ASSERT_TRUE(CallOnResponseStarted()); 944 ASSERT_TRUE(CallOnResponseStarted());
943 EXPECT_TRUE(url_loader_client_.has_received_upload_progress()); 945 EXPECT_TRUE(url_loader_client_.has_received_upload_progress());
944 EXPECT_EQ(1000, url_loader_client_.current_upload_position()); 946 EXPECT_EQ(1000, url_loader_client_.current_upload_position());
945 EXPECT_EQ(1000, url_loader_client_.total_upload_size()); 947 EXPECT_EQ(1000, url_loader_client_.total_upload_size());
946 } 948 }
947 949
950 TEST_F(MojoAsyncResourceHandlerTest, SetPriority) {
951 ASSERT_TRUE(CallOnWillStartAndOnResponseStarted());
952 std::unique_ptr<ResourceThrottle> throttle =
953 ResourceDispatcherHostImpl::Get()->scheduler()->ScheduleRequest(
954 4, 2, false, request_.get());
955
956 EXPECT_EQ(net::LOWEST, request_->priority());
957
958 handler_->SetPriority(mojom::RequestPriority::kHighest, 4);
959
960 EXPECT_EQ(net::HIGHEST, request_->priority());
961 }
962
948 TEST_P(MojoAsyncResourceHandlerWithAllocationSizeTest, 963 TEST_P(MojoAsyncResourceHandlerWithAllocationSizeTest,
949 OnWillReadWithLongContents) { 964 OnWillReadWithLongContents) {
950 ASSERT_TRUE(CallOnWillStartAndOnResponseStarted()); 965 ASSERT_TRUE(CallOnWillStartAndOnResponseStarted());
951 ASSERT_EQ(MockResourceLoader::Status::IDLE, mock_loader_->OnWillRead(-1)); 966 ASSERT_EQ(MockResourceLoader::Status::IDLE, mock_loader_->OnWillRead(-1));
952 std::string expected; 967 std::string expected;
953 for (int i = 0; i < 3 * mock_loader_->io_buffer_size() + 2; ++i) 968 for (int i = 0; i < 3 * mock_loader_->io_buffer_size() + 2; ++i)
954 expected += ('A' + i % 26); 969 expected += ('A' + i % 26);
955 970
956 ASSERT_EQ(MockResourceLoader::Status::IDLE, mock_loader_->OnReadCompleted(0)); 971 ASSERT_EQ(MockResourceLoader::Status::IDLE, mock_loader_->OnReadCompleted(0));
957 972
(...skipping 304 matching lines...) Expand 10 before | Expand all | Expand 10 after
1262 } 1277 }
1263 } 1278 }
1264 EXPECT_EQ("B", body); 1279 EXPECT_EQ("B", body);
1265 } 1280 }
1266 1281
1267 INSTANTIATE_TEST_CASE_P(MojoAsyncResourceHandlerWithAllocationSizeTest, 1282 INSTANTIATE_TEST_CASE_P(MojoAsyncResourceHandlerWithAllocationSizeTest,
1268 MojoAsyncResourceHandlerWithAllocationSizeTest, 1283 MojoAsyncResourceHandlerWithAllocationSizeTest,
1269 ::testing::Values(8, 32 * 2014)); 1284 ::testing::Values(8, 32 * 2014));
1270 } // namespace 1285 } // namespace
1271 } // namespace content 1286 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698