Linux: Breakpad updates.

Linux: Breakpad updates.

The crash server requires an MDRawSystemInfo stream in order to process the
file, although it actually only needs a single value in it.

Additionally, it needs a PDB7 structure for each module, so we remove those
modules which probably aren't shared libraries and XOR the first page of data
to get a signature for them. The previous code opened the file and MD5 summed
the text segment. However, we can't know if the shared library is still linked
on disk, nor if the whole text segment is mapped in memory. Thus, we only
assume that a single page is mapped.

dump_syms will need to be forked and updated to match the signature scheme used
in this patch.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=17030

[Lei Zhang, 2009-05-27 00:06:19]

LGTM

http://codereview.chromium.org/113875/diff/1/4
File breakpad/linux/minidump_writer.cc (right):

http://codereview.chromium.org/113875/diff/1/4#newcode508
Line 508: continue;
Can we put this and the duplicate code above in its own function?

[Lei Zhang, 2009-05-27 07:10:55]

LGTM

http://codereview.chromium.org/113875/diff/1007/7
File breakpad/linux/minidump_writer.cc (right):

http://codereview.chromium.org/113875/diff/1007/7#newcode477
Line 477: mapping.size < 4096) { // too small to get a signature for.
style nit: two space before //