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

Unified Diff: components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc

Issue 2889993004: New CPAT support in DataReductionProxyConfig guarded by feature flag. (Closed)
Patch Set: Updates new code path to accept LoFi when LitePages enabled (to allow fallback) Created 3 years, 7 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 side-by-side diff with in-line comments
Download patch
Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
index 5d7b2ba89628098afab5691934805e5a39612b93..4465b2b6af3248c115444a68ec4ae2fd5b3aa34d 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc
@@ -888,7 +888,7 @@ TEST_F(DataReductionProxyNetworkDelegateTest, LoFiTransitions) {
GURL(kTestURL), net::IDLE, &delegate, TRAFFIC_ANNOTATION_FOR_TESTS);
fake_request->SetLoadFlags(net::LOAD_MAIN_FRAME_DEPRECATED);
lofi_decider()->SetIsUsingLoFi(config()->ShouldEnableLoFi(
- *fake_request.get(), &test_previews_decider));
+ *fake_request.get(), test_previews_decider));
NotifyNetworkDelegate(fake_request.get(), data_reduction_proxy_info,
proxy_retry_info, &headers);
@@ -896,7 +896,7 @@ TEST_F(DataReductionProxyNetworkDelegateTest, LoFiTransitions) {
VerifyDataReductionProxyData(
*fake_request, tests[i].is_data_reduction_proxy,
config()->ShouldEnableLoFi(*fake_request.get(),
- &test_previews_decider));
+ test_previews_decider));
}
{
@@ -971,13 +971,13 @@ TEST_F(DataReductionProxyNetworkDelegateTest, LoFiTransitions) {
GURL(kTestURL), net::IDLE, &delegate, TRAFFIC_ANNOTATION_FOR_TESTS);
fake_request->SetLoadFlags(net::LOAD_MAIN_FRAME_DEPRECATED);
lofi_decider()->SetIsUsingLoFi(config()->ShouldEnableLoFi(
- *fake_request.get(), &test_previews_decider));
+ *fake_request.get(), test_previews_decider));
NotifyNetworkDelegate(fake_request.get(), data_reduction_proxy_info,
proxy_retry_info, &headers);
VerifyDataReductionProxyData(
*fake_request, tests[i].is_data_reduction_proxy,
config()->ShouldEnableLoFi(*fake_request.get(),
- &test_previews_decider));
+ test_previews_decider));
}
}
}
@@ -1270,8 +1270,8 @@ TEST_F(DataReductionProxyNetworkDelegateTest, NetHistograms) {
// Needed as a parameter, but functionality is not tested.
TestPreviewsDecider test_previews_decider;
- lofi_decider()->SetIsUsingLoFi(config()->ShouldEnableLoFi(
- *fake_request.get(), &test_previews_decider));
+ lofi_decider()->SetIsUsingLoFi(
+ config()->ShouldEnableLoFi(*fake_request.get(), test_previews_decider));
fake_request = (FetchURLRequest(GURL(kTestURL), nullptr, response_headers,
kResponseContentLength, 0));

Powered by Google App Engine
This is Rietveld 408576698