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

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

Issue 2572453002: [PlzNavigate] Wire page visibility info to navigation requests (Closed)
Patch Set: Created 4 years 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
« no previous file with comments | « content/browser/loader/resource_dispatcher_host_impl.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 <stddef.h> 5 #include <stddef.h>
6 6
7 #include <memory> 7 #include <memory>
8 #include <utility> 8 #include <utility>
9 #include <vector> 9 #include <vector>
10 10
(...skipping 1069 matching lines...) Expand 10 before | Expand all | Expand 10 after
1080 if (IsBrowserSideNavigationEnabled()) { 1080 if (IsBrowserSideNavigationEnabled()) {
1081 auto_advance_ = true; 1081 auto_advance_ = true;
1082 1082
1083 // Make a navigation request. 1083 // Make a navigation request.
1084 TestNavigationURLLoaderDelegate delegate; 1084 TestNavigationURLLoaderDelegate delegate;
1085 BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL, false, 1085 BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL, false,
1086 REQUEST_CONTEXT_TYPE_LOCATION); 1086 REQUEST_CONTEXT_TYPE_LOCATION);
1087 CommonNavigationParams common_params; 1087 CommonNavigationParams common_params;
1088 common_params.url = url; 1088 common_params.url = url;
1089 std::unique_ptr<NavigationRequestInfo> request_info( 1089 std::unique_ptr<NavigationRequestInfo> request_info(
1090 new NavigationRequestInfo(common_params, begin_params, url, 1090 new NavigationRequestInfo(
1091 url::Origin(url), true, false, false, -1, 1091 common_params, begin_params, url, url::Origin(url), true, false,
1092 false, false)); 1092 false, -1, false, false, blink::WebPageVisibilityStateVisible));
1093 std::unique_ptr<NavigationURLLoader> test_loader = 1093 std::unique_ptr<NavigationURLLoader> test_loader =
1094 NavigationURLLoader::Create(browser_context_.get(), 1094 NavigationURLLoader::Create(browser_context_.get(),
1095 std::move(request_info), nullptr, nullptr, 1095 std::move(request_info), nullptr, nullptr,
1096 nullptr, &delegate); 1096 nullptr, &delegate);
1097 1097
1098 // The navigation should fail with the expected error code. 1098 // The navigation should fail with the expected error code.
1099 delegate.WaitForRequestFailed(); 1099 delegate.WaitForRequestFailed();
1100 ASSERT_EQ(expected_error_code, delegate.net_error()); 1100 ASSERT_EQ(expected_error_code, delegate.net_error());
1101 return; 1101 return;
1102 } 1102 }
(...skipping 1536 matching lines...) Expand 10 before | Expand all | Expand 10 after
2639 if (IsBrowserSideNavigationEnabled()) { 2639 if (IsBrowserSideNavigationEnabled()) {
2640 // Create a NavigationRequest. 2640 // Create a NavigationRequest.
2641 TestNavigationURLLoaderDelegate delegate; 2641 TestNavigationURLLoaderDelegate delegate;
2642 BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL, false, 2642 BeginNavigationParams begin_params(std::string(), net::LOAD_NORMAL, false,
2643 REQUEST_CONTEXT_TYPE_LOCATION); 2643 REQUEST_CONTEXT_TYPE_LOCATION);
2644 CommonNavigationParams common_params; 2644 CommonNavigationParams common_params;
2645 common_params.url = download_url; 2645 common_params.url = download_url;
2646 std::unique_ptr<NavigationRequestInfo> request_info( 2646 std::unique_ptr<NavigationRequestInfo> request_info(
2647 new NavigationRequestInfo(common_params, begin_params, download_url, 2647 new NavigationRequestInfo(common_params, begin_params, download_url,
2648 url::Origin(download_url), true, false, false, 2648 url::Origin(download_url), true, false, false,
2649 -1, false, false)); 2649 -1, false, false,
2650 blink::WebPageVisibilityStateVisible));
2650 std::unique_ptr<NavigationURLLoader> loader = NavigationURLLoader::Create( 2651 std::unique_ptr<NavigationURLLoader> loader = NavigationURLLoader::Create(
2651 browser_context_.get(), std::move(request_info), nullptr, nullptr, 2652 browser_context_.get(), std::move(request_info), nullptr, nullptr,
2652 nullptr, &delegate); 2653 nullptr, &delegate);
2653 2654
2654 // Wait until a response has been received and proceed with the response. 2655 // Wait until a response has been received and proceed with the response.
2655 KickOffRequest(); 2656 KickOffRequest();
2656 2657
2657 // Return some data so that the request is identified as a download 2658 // Return some data so that the request is identified as a download
2658 // and the proper resource handlers are created. 2659 // and the proper resource handlers are created.
2659 EXPECT_TRUE(net::URLRequestTestJob::ProcessOnePendingMessage()); 2660 EXPECT_TRUE(net::URLRequestTestJob::ProcessOnePendingMessage());
(...skipping 1328 matching lines...) Expand 10 before | Expand all | Expand 10 after
3988 return nullptr; 3989 return nullptr;
3989 } 3990 }
3990 3991
3991 INSTANTIATE_TEST_CASE_P( 3992 INSTANTIATE_TEST_CASE_P(
3992 ResourceDispatcherHostTests, 3993 ResourceDispatcherHostTests,
3993 ResourceDispatcherHostTest, 3994 ResourceDispatcherHostTest,
3994 testing::Values(TestConfig::kDefault, 3995 testing::Values(TestConfig::kDefault,
3995 TestConfig::kOptimizeIPCForSmallResourceEnabled)); 3996 TestConfig::kOptimizeIPCForSmallResourceEnabled));
3996 3997
3997 } // namespace content 3998 } // namespace content
OLDNEW
« no previous file with comments | « content/browser/loader/resource_dispatcher_host_impl.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698