[microdump] Fix hw architecture indication in build fingerprint line

[microdump] Fix hw architecture indication in build fingerprint line

r1456 introduced the possibility to customize the OS-line of the
microdump, enabling to replace, in the case of android, the generic
uname() info with the Android build fingerprint.
While doing that, it mistakenly removed the HW architecture indication
from the format.
See crbug.com/520075 for more details.

BUG=chromium:520075
R=mmandlis@chromium.org, torne@chromium.org

Committed: https://code.google.com/p/google-breakpad/source/detail?r=1489

[Primiano Tucci (use gerrit), 2015-08-13 09:15:35]

primiano@chromium.org changed reviewers:
+ mmandlis@chromium.org, torne@chromium.org

[Torne, 2015-08-13 09:34:22]

lgtm

[mmandlis, 2015-08-13 16:31:24]

https://codereview.chromium.org/1288313002/diff/1/src/client/linux/microdump_...
File src/client/linux/microdump_writer/microdump_writer.cc (right):

https://codereview.chromium.org/1288313002/diff/1/src/client/linux/microdump_...
src/client/linux/microdump_writer/microdump_writer.cc:210: LogAppend("no build
fingerprint available");
could you, please, add tests for this code?

[Primiano Tucci (use gerrit), 2015-08-14 11:46:51]

Patchset #3 (id:40001) has been deleted

[Primiano Tucci (use gerrit), 2015-08-14 11:46:57]

Patchset #2 (id:20001) has been deleted

[Primiano Tucci (use gerrit), 2015-08-14 11:47:17]

Patchset #2 (id:60001) has been deleted

[Primiano Tucci (use gerrit), 2015-08-14 11:49:32]

https://codereview.chromium.org/1288313002/diff/1/src/client/linux/microdump_...
File src/client/linux/microdump_writer/microdump_writer.cc (right):

https://codereview.chromium.org/1288313002/diff/1/src/client/linux/microdump_...
src/client/linux/microdump_writer/microdump_writer.cc:210: LogAppend("no build
fingerprint available");
On 2015/08/13 16:31:24, mmandlis wrote:
> could you, please, add tests for this code?

Done.

[Primiano Tucci (use gerrit), 2015-08-14 16:28:44]

Maria, can you take a second look to this plz (added tests)?

[mmandlis, 2015-08-14 16:45:06]

lgtm

Thanks!

[Primiano Tucci (use gerrit), 2015-08-17 08:02:27]

Committed patchset #2 (id:80001) manually as 1489 (presubmit successful).