Add MinidumpStringListWriter and test

Add MinidumpStringListWriter and test.

TEST=minidump_test MinidumpStringWriter.*List
R=rsesek@chromium.org

Committed: https://chromium.googlesource.com/crashpad/crashpad/+/6e97189d2733cb04b3352204b80d10b134fe8ba6

[Mark Mentovai, 2014-11-06 19:37:41]

mark@chromium.org changed reviewers:
+ rsesek@chromium.org

[Mark Mentovai, 2014-11-06 19:37:41]

This depends on https://codereview.chromium.org/704333002/.

[Mark Mentovai, 2014-11-06 19:54:28]

Patchset #2 (id:20001) has been deleted

[Robert Sesek, 2014-11-07 14:47:01]

LGTM

https://codereview.chromium.org/707003002/diff/40001/minidump/minidump_string...
File minidump/minidump_string_writer_test.cc (right):

https://codereview.chromium.org/707003002/diff/40001/minidump/minidump_string...
minidump/minidump_string_writer_test.cc:221: template <typename Traits>
There's gtest support for this, too:
https://code.google.com/p/googletest/wiki/AdvancedGuide#Typed_Tests

[Mark Mentovai, 2014-11-07 14:56:49]

https://codereview.chromium.org/707003002/diff/40001/minidump/minidump_string...
File minidump/minidump_string_writer_test.cc (right):

https://codereview.chromium.org/707003002/diff/40001/minidump/minidump_string...
minidump/minidump_string_writer_test.cc:221: template <typename Traits>
Robert Sesek wrote:
> There's gtest support for this, too:
> https://code.google.com/p/googletest/wiki/AdvancedGuide#Typed_Tests

I’ll keep that in mind for the future, but I don’t think it works as well in
this instance. It looks like that will repeat the entire test case for the
desired types, but I only want to repeat a single test. I don’t think that it’s
an advantage to break out a new test case for this.

[Mark Mentovai, 2014-11-07 14:57:15]

Committed patchset #3 (id:60001) manually as
6e97189d2733cb04b3352204b80d10b134fe8ba6 (presubmit successful).