Data reduction proxy bypass with test server

net/test/embedded_test_server/embedded_test_server.cc

Index: net/test/embedded_test_server/embedded_test_server.cc
diff --git a/net/test/embedded_test_server/embedded_test_server.cc b/net/test/embedded_test_server/embedded_test_server.cc
index 97b5d84509c5f2324872ce69c3bfedf9459c582e..7a44803b882e98d69f7f4b10a453ada517dd582b 100644
--- a/net/test/embedded_test_server/embedded_test_server.cc
+++ b/net/test/embedded_test_server/embedded_test_server.cc
@@ -49,8 +49,14 @@ scoped_ptr<HttpResponse> HandleFileRequest(
   // This is a test-only server. Ignore I/O thread restrictions.
   base::ThreadRestrictions::ScopedAllowIO allow_io;
 
+  std::string relative_url(request.relative_url);
+  // A proxy request will have an absolute path. Simulate the proxy by stripping
+  // the scheme, host, and port.
+  if (GURL(relative_url).is_valid())

[mmenke, 2014/12/08 18:16:05]

nit:  Should create the GURL on the stack, to avoid the duplicate GURL
constructor call (Admittedly, performance doesn't matter at all here, but
creating a GURL twice is just not needed)

[bengr, 2014/12/08 22:29:26]

Done.

+    relative_url = GURL(relative_url).PathForRequest();
+
   // Trim the first byte ('/').
-  std::string request_path(request.relative_url.substr(1));
+  std::string request_path = relative_url.substr(1);
 
   // Remove the query string if present.
   size_t query_pos = request_path.find('?');