Make URLRequestTestJob accept \n terminated header lines.

components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc

Index: components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
diff --git a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
index cf21046949e2517e7ab4221f1ce625cb157d5f1d..6a01642506e89b73098406eeee2a55d8ee4aa3f0 100644
--- a/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
+++ b/components/data_reduction_proxy/core/browser/data_reduction_proxy_bypass_stats_unittest.cc
@@ -411,7 +411,6 @@ TEST_F(DataReductionProxyBypassStatsTest, RecordMissingViaHeaderBytes) {
     scoped_ptr<DataReductionProxyBypassStats> bypass_stats = BuildBypassStats();
 
     std::string raw_headers(test_cases[i].headers);

[mmenke, 2015/10/27 16:49:31]

nit:  Can just inline test_cases[i].headers below.

[martijnc, 2015/10/27 21:10:15]

Done.

-    HeadersToRaw(&raw_headers);
 
     scoped_ptr<net::URLRequest> fake_request(
         CreateURLRequestWithResponseHeaders(GURL("http://www.google.com/"),
@@ -479,7 +478,6 @@ TEST_F(DataReductionProxyBypassStatsTest, RequestCompletionErrorCodes) {
 
     std::string raw_headers("HTTP/1.1 200 OK\n"
                             "Via: 1.1 Chrome-Compression-Proxy\n");

[mmenke, 2015/10/27 16:49:31]

nit:  Remove "raw" (I suggest adding response instead, to match
CreateURLRequestWithResponseHeaders's prototype, but that's optional).

[martijnc, 2015/10/27 21:10:15]

Done.

-    HeadersToRaw(&raw_headers);
     scoped_ptr<net::URLRequest> fake_request(
         CreateURLRequestWithResponseHeaders(GURL("http://www.google.com/"),
                                             raw_headers));