New CPAT support in DataReductionProxyConfig guarded by feature flag.

components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate_unittest.cc

 // Copyright 2014 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 "components/data_reduction_proxy/core/browser/data_reduction_proxy_network_delegate.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <map>
@@ skipping 870 matching lines @@
     {
       // Main frame loaded. Lo-Fi should be used.
       net::HttpRequestHeaders headers;
       net::ProxyRetryInfoMap proxy_retry_info;
 
       net::TestDelegate delegate;
       std::unique_ptr<net::URLRequest> fake_request = context()->CreateRequest(
           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);
 
       VerifyHeaders(tests[i].is_data_reduction_proxy, true, headers);
       VerifyDataReductionProxyData(
           *fake_request, tests[i].is_data_reduction_proxy,
           config()->ShouldEnableLoFi(*fake_request.get(),
-                                     &test_previews_decider));
+                                     test_previews_decider));
     }
 
     {
       // Lo-Fi is already off. Lo-Fi should not be used.
       net::HttpRequestHeaders headers;
       net::ProxyRetryInfoMap proxy_retry_info;
       net::TestDelegate delegate;
       std::unique_ptr<net::URLRequest> fake_request = context()->CreateRequest(
           GURL(kTestURL), net::IDLE, &delegate, TRAFFIC_ANNOTATION_FOR_TESTS);
       lofi_decider()->SetIsUsingLoFi(false);
@@ skipping 54 matching lines @@
 
     {
       // Main frame request. Lo-Fi should be used.
       net::HttpRequestHeaders headers;
       net::ProxyRetryInfoMap proxy_retry_info;
       net::TestDelegate delegate;
       std::unique_ptr<net::URLRequest> fake_request = context()->CreateRequest(
           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));
     }
   }
 }
 
 TEST_F(DataReductionProxyNetworkDelegateTest, RequestDataConfigurations) {
   Init(USE_INSECURE_PROXY, false);
   const struct {
     bool lofi_on;
     bool used_data_reduction_proxy;
     bool main_frame;
@@ skipping 272 matching lines @@
     }
 
     if (tests[i].lofi_enabled_through_switch) {
       base::CommandLine::ForCurrentProcess()->AppendSwitchASCII(
           switches::kDataReductionProxyLoFi,
           switches::kDataReductionProxyLoFiValueAlwaysOn);
     }
 
     // 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));
     fake_request->SetLoadFlags(fake_request->load_flags() |
                                net::LOAD_MAIN_FRAME_DEPRECATED);
 
     // Histograms are accumulative, so get the sum of all the tests so far.
     int expected_count = 0;
     for (size_t j = 0; j <= i; ++j)
       expected_count += tests[j].expected_count;
@@ skipping 590 matching lines @@
   EXPECT_EQ(static_cast<int64_t>(net::HttpUtil::AssembleRawHeaders(
                                      kHeaders, sizeof(kHeaders) - 1)
                                      .size() +
                                  10000 - request->GetTotalReceivedBytes()),
             GetSavings());
 }
 
 }  // namespace
 
 }  // namespace data_reduction_proxy