HTTPTransport test: Deal with limited-size pipe buffers

util/net/http_transport_test.cc

Index: util/net/http_transport_test.cc
diff --git a/util/net/http_transport_test.cc b/util/net/http_transport_test.cc
index 25a1a43a3264bf2f84888828208fedff87e4a3e3..0bb2985738a21e69901c633dd5ea1a56bb1c790f 100644
--- a/util/net/http_transport_test.cc
+++ b/util/net/http_transport_test.cc
@@ -278,7 +278,7 @@ TEST(HTTPTransport, UnchunkedPlainText) {
   test.Run();
 }
 
-TEST(HTTPTransport, Upload33k) {
+void RunUpload33k(bool has_content_length) {
   // On OS X, NSMutableURLRequest winds up calling into a CFReadStream’s Read()
   // callback with a 32kB buffer. Make sure that it’s able to get everything
   // when enough is available to fill this buffer, requiring more than one
@@ -290,8 +290,10 @@ TEST(HTTPTransport, Upload33k) {
 
   HTTPHeaders headers;
   headers[kContentType] = "application/octet-stream";
-  headers[kContentLength] =
-      base::StringPrintf("%" PRIuS, request_string.size());
+  if (has_content_length) {
+    headers[kContentLength] =
+        base::StringPrintf("%" PRIuS, request_string.size());
+  }
   HTTPTransportTestFixture test(headers, body_stream.Pass(), 200,
       [](HTTPTransportTestFixture* fixture, const std::string& request) {
         size_t body_start = request.rfind("\r\n");
@@ -300,6 +302,15 @@ TEST(HTTPTransport, Upload33k) {
   test.Run();
 }
 
+TEST(HTTPTransport, Upload33k) {
+  RunUpload33k(true);
+}
+
+TEST(HTTPTransport, Upload33k_LengthUnknown) {
+  // The same as Upload33k, but without declaring Content-Length ahead of time.
+  RunUpload33k(false);
+}
+
 }  // namespace
 }  // namespace test
 }  // namespace crashpad