minidump: Migrate the rest of the tests to MinidumpWritableAtLocationDescriptor<>()

minidump: Migrate the rest of the tests to
MinidumpWritableAtLocationDescriptor<>().

TEST=minidump_test
R=rsesek@chromium.org

Committed: https://chromium.googlesource.com/crashpad/crashpad/+/20f95a2f5bb2500567328c2c1b3793f6a918cfac

[Mark Mentovai, 2014-10-21 21:40:12]

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

[Robert Sesek, 2014-10-22 21:57:52]

LGTM

https://codereview.chromium.org/670853002/diff/1/minidump/test/minidump_writa...
File minidump/test/minidump_writable_test_util.cc (right):

https://codereview.chromium.org/670853002/diff/1/minidump/test/minidump_writa...
minidump/test/minidump_writable_test_util.cc:93: if (misc->Length % 2 != 0) {
Would be helpful to have a comment that Unicode indicates UTF-16/2-byte code
units.

https://codereview.chromium.org/670853002/diff/1/minidump/test/minidump_writa...
minidump/test/minidump_writable_test_util.cc:166: static size_t
ElementCount(const ListType* list) { return list->count; }
nit: This is the only impl that you onelined.

[Mark Mentovai, 2014-10-22 22:35:23]

Committed patchset #2 (id:20001) manually as
20f95a2f5bb2500567328c2c1b3793f6a918cfac (presubmit successful).