Adding unit test for two new functions in mime_util.h/cc.

net/base/mime_util_unittest.cc

Index: net/base/mime_util_unittest.cc
diff --git a/net/base/mime_util_unittest.cc b/net/base/mime_util_unittest.cc
index 565f9deeec7e44c2bf540343a7b2955866830065..079953be1c448db6d94a27acb65858cf1284777e 100644
--- a/net/base/mime_util_unittest.cc
+++ b/net/base/mime_util_unittest.cc
@@ -296,4 +296,29 @@ TEST(MimeUtilTest, TestGetCertificateMimeTypeForMimeType) {
             GetCertificateMimeTypeForMimeType("text/plain"));
 }
 
+TEST(MimeUtilTest, TestAddMultipartValueForUpload) {
+  const char* ref_output = "--boundary\r\nContent-Disposition: form-data;"
+                           " name=\"value name\"\r\nContent-Type: content type"
+                           "\r\n\r\nvalue\r\n";
+  std::string post_data;
+  AddMultipartValueForUpload("value name", "value", "boundary",
+                             "content type", &post_data);
+  EXPECT_STREQ(ref_output, post_data.c_str());
+
+  const char* ref_output_no_content_type =
+      "--boundary\r\nContent-Disposition: form-data;"
+      " name=\"value name\"\r\n\r\nvalue\r\n";
+  post_data.clear();

[wtc, 2013/05/16 18:24:22]

Nit: it would be better to test two AddMultipartValueForUpload
calls, without this post_data.clear() call in between,
and just check the combined result once.

I would probably also include the AddMultipartFinalDelimiterForUpload
call in this unit test and omit the next unit test.

[Henrik Grunell, 2013/05/17 07:49:12]

Done.

+  AddMultipartValueForUpload("value name", "value", "boundary",
+                             "", &post_data);
+  EXPECT_STREQ(ref_output_no_content_type, post_data.c_str());
+}
+
+TEST(MimeUtilTest, TestAddMultipartFinalDelimiterForUpload) {
+  const char* ref_output = "--boundary--\r\n";
+  std::string post_data;
+  AddMultipartFinalDelimiterForUpload("boundary", &post_data);
+  EXPECT_STREQ(ref_output, post_data.c_str());
+}
+
 }  // namespace net