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

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

New functions were added in
https://chromiumcodereview.appspot.com/15076008/

BUG=229829
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=200789

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

Patch set 1 LGTM. Thanks.

https://codereview.chromium.org/14934008/diff/1/net/base/mime_util_unittest.cc
File net/base/mime_util_unittest.cc (right):

https://codereview.chromium.org/14934008/diff/1/net/base/mime_util_unittest.c...
net/base/mime_util_unittest.cc:311: post_data.clear();

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]

https://chromiumcodereview.appspot.com/14934008/diff/1/net/base/mime_util_uni...
File net/base/mime_util_unittest.cc (right):

https://chromiumcodereview.appspot.com/14934008/diff/1/net/base/mime_util_uni...
net/base/mime_util_unittest.cc:311: post_data.clear();
On 2013/05/16 18:24:22, wtc wrote:
> 
> 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.

Done.

[commit-bot: I haz the power, 2013-05-17 07:49:29]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/grunell@chromium.org/14934008/5001

[commit-bot: I haz the power, 2013-05-17 11:38:07]

Change committed as 200789