Added chunked upload support for embedded_test_server

net/test/embedded_test_server/http_request_unittest.cc

Index: net/test/embedded_test_server/http_request_unittest.cc
diff --git a/net/test/embedded_test_server/http_request_unittest.cc b/net/test/embedded_test_server/http_request_unittest.cc
index 56121bbc8e60e5a626177b325f64ba0a02583ea8..61d4d2115732c506bb044a1dfd105bc8a315931a 100644
--- a/net/test/embedded_test_server/http_request_unittest.cc
+++ b/net/test/embedded_test_server/http_request_unittest.cc
@@ -76,6 +76,40 @@ TEST(HttpRequestTest, ParseRequestWithEmptyBody) {
   EXPECT_EQ("0", request->headers["Content-Length"]);
 }
 
+TEST(HttpRequestTest, ParseRequestWithChunkedBody) {
+  HttpRequestParser parser;
+
+  parser.ProcessChunk("POST /foobar.html HTTP/1.1\r\n");
+  parser.ProcessChunk("Transfer-Encoding: chunked\r\n\r\n");
+  parser.ProcessChunk("5\r\nhello\r\n");
+  parser.ProcessChunk("1\r\n \r\n");
+  parser.ProcessChunk("5\r\nworld\r\n");
+  parser.ProcessChunk("0\r\n\r\n");
+  ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
+
+  scoped_ptr<HttpRequest> request = parser.GetRequest();

[mmenke, 2015/02/13 17:53:29]

Should include base/memory/scoped_ptr (File already depends on it, but it's not
explicitly being included)

[xunjieli, 2015/02/13 18:36:14]

Done.

+  EXPECT_EQ("hello world", request->content);
+  EXPECT_TRUE(request->has_content);
+  EXPECT_EQ(1u, request->headers.count("Transfer-Encoding"));
+  EXPECT_EQ("chunked", request->headers["Transfer-Encoding"]);
+}
+
+TEST(HttpRequestTest, ParseRequestWithChunkedBodySlow) {
+  HttpRequestParser parser;
+
+  parser.ProcessChunk("POST /foobar.html HTTP/1.1\r\n");
+  parser.ProcessChunk("Transfer-Encoding: chunked\r\n\r\n");
+  parser.ProcessChunk("5\r\nhello\r\n");
+  ASSERT_EQ(HttpRequestParser::WAITING, parser.ParseRequest());
+  parser.ProcessChunk("0\r\n\r\n");

[mmenke, 2015/02/13 17:53:29]

You need to send partial chunks here, not entire chunks at once.  That was the
whole point processing the response character-by-character.

[xunjieli, 2015/02/13 18:36:14]

Right, that's why I called parser.ParseRequest() on line 103, and again on line
105. Should I stop at line 104 then? 

[mmenke, 2015/02/13 18:43:29]

You're still calling it at chunk boundaries...So if you're using the chunked
decoder incorrectly, but in a way that works as long as you only call it at
chunk boundaries, this test would incorrectly pass.  Should append data one
character at a time and then call ParseRequest, to make sure sure everything
works (In my earlier example, hadn't realized ProcessChunk doesn't call into
ParseRequest)

+  ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
+  scoped_ptr<HttpRequest> request = parser.GetRequest();
+  EXPECT_EQ("hello", request->content);
+  EXPECT_TRUE(request->has_content);
+  EXPECT_EQ(1u, request->headers.count("Transfer-Encoding"));
+  EXPECT_EQ("chunked", request->headers["Transfer-Encoding"]);
+}
+
 TEST(HttpRequestTest, ParseRequestWithoutBody) {
   HttpRequestParser parser;