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

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

Issue 2374863002: Remove usage of HostZoomMap from c/b/loader via ReadyToCommitNavigation (Closed)
Patch Set: init Created 4 years, 1 month 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/url_loader_factory_impl.h" 5 #include "content/browser/loader/url_loader_factory_impl.h"
6 6
7 #include <memory> 7 #include <memory>
8 #include <string> 8 #include <string>
9 #include <utility> 9 #include <utility>
10 10
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 // If browser side navigation is enabled then 83 // If browser side navigation is enabled then
84 // ResourceDispatcherHostImpl prevents main frame URL requests from 84 // ResourceDispatcherHostImpl prevents main frame URL requests from
85 // the renderer. Ensure that these checks don't trip us up by 85 // the renderer. Ensure that these checks don't trip us up by
86 // setting the process type in ResourceMessageFilter as 86 // setting the process type in ResourceMessageFilter as
87 // PROCESS_TYPE_UNKNOWN. 87 // PROCESS_TYPE_UNKNOWN.
88 PROCESS_TYPE_UNKNOWN, 88 PROCESS_TYPE_UNKNOWN,
89 nullptr, 89 nullptr,
90 nullptr, 90 nullptr,
91 nullptr, 91 nullptr,
92 nullptr, 92 nullptr,
93 nullptr,
94 base::Bind(&URLLoaderFactoryImplTest::GetContexts, 93 base::Bind(&URLLoaderFactoryImplTest::GetContexts,
95 base::Unretained(this)))) { 94 base::Unretained(this)))) {
96 MojoAsyncResourceHandler::SetAllocationSizeForTesting(GetParam()); 95 MojoAsyncResourceHandler::SetAllocationSizeForTesting(GetParam());
97 rdh_.SetLoaderDelegate(&loader_deleate_); 96 rdh_.SetLoaderDelegate(&loader_deleate_);
98 97
99 URLLoaderFactoryImpl::Create(resource_message_filter_, 98 URLLoaderFactoryImpl::Create(resource_message_filter_,
100 mojo::GetProxy(&factory_)); 99 mojo::GetProxy(&factory_));
101 100
102 // Calling this function creates a request context. 101 // Calling this function creates a request context.
103 browser_context_->GetResourceContext()->GetRequestContext(); 102 browser_context_->GetResourceContext()->GetRequestContext();
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
260 EXPECT_EQ(net::ERR_ABORTED, client.completion_status().error_code); 259 EXPECT_EQ(net::ERR_ABORTED, client.completion_status().error_code);
261 } 260 }
262 261
263 INSTANTIATE_TEST_CASE_P(URLLoaderFactoryImplTest, 262 INSTANTIATE_TEST_CASE_P(URLLoaderFactoryImplTest,
264 URLLoaderFactoryImplTest, 263 URLLoaderFactoryImplTest,
265 ::testing::Values(128, 32 * 1024)); 264 ::testing::Values(128, 32 * 1024));
266 265
267 } // namespace 266 } // namespace
268 267
269 } // namespace content 268 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_message_filter.cc ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698