Update stability report dumper so it handles non-postmortem minidumps

Update stability report dumper so it handles non-postmortem minidumps

BUG=718437
Review-Url: https://codereview.chromium.org/2866163002
Cr-Commit-Position: refs/heads/master@{#470428}
Committed: https://chromium.googlesource.com/chromium/src/+/85e629e964e7da6d8a3a05b713e2aef62f976595

[manzagop (departed), 2017-05-08 22:23:01]

manzagop@chromium.org changed reviewers:
+ bcwhite@chromium.org

[manzagop (departed), 2017-05-08 22:23:02]

This revises the dumper prior to the crashpad integration. Please have a look.
Thanks!
Pierre

[manzagop (departed), 2017-05-08 22:26:54]

The CQ bit was checked by manzagop@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-08 22:30:45]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-08 23:28:06]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-08 23:28:07]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[manzagop (departed), 2017-05-09 12:23:48]

The CQ bit was checked by manzagop@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-09 12:24:05]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-09 12:32:59]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-09 12:33:03]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[manzagop (departed), 2017-05-09 14:14:07]

The CQ bit was checked by manzagop@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-09 14:14:24]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[manzagop (departed), 2017-05-09 14:16:45]

manzagop@chromium.org changed reviewers:
+ dpranke@chromium.org

[manzagop (departed), 2017-05-09 14:16:46]

Hi Dirk,

Could you have a look at the PRESUBMIT changes? 
1) I'm trying to add to add a file to the log spam exclusion list (it's just a
tool). 
2) That modification triggers a CheckAddedDepsHaveTetsApprovalsTest failure due
to being on windows and the paths, so I sprinkled some os.path.join. 

Thanks!
Pierre

https://codereview.chromium.org/2866163002/diff/40001/PRESUBMIT_test.py
File PRESUBMIT_test.py (right):

https://codereview.chromium.org/2866163002/diff/40001/PRESUBMIT_test.py#newco...
PRESUBMIT_test.py:505: '+grit/generated_resources.h",',
Quick validation: is the ", expected there? Same in a few other places.

[bcwhite, 2017-05-09 14:33:47]

lgtm

[commit-bot: I haz the power, 2017-05-09 15:30:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-09 15:30:37]

Dry run: This issue passed the CQ dry run.

[Dirk Pranke, 2017-05-09 20:23:45]

lgtm

[manzagop (departed), 2017-05-09 20:45:44]

Thanks for the reviews!

[manzagop (departed), 2017-05-09 20:45:49]

The CQ bit was checked by manzagop@chromium.org

[commit-bot: I haz the power, 2017-05-09 20:46:57]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-09 22:11:51]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494362749698830,
"parent_rev": "830937ec4ec791325c178c119566b6ec88632449", "commit_rev":
"85e629e964e7da6d8a3a05b713e2aef62f976595"}

[commit-bot: I haz the power, 2017-05-09 22:12:04]

Description was changed from

==========
Update stability report dumper so it handles non-postmortem minidumps

BUG=718437
==========

to

==========
Update stability report dumper so it handles non-postmortem minidumps

BUG=718437

Review-Url: https://codereview.chromium.org/2866163002
Cr-Commit-Position: refs/heads/master@{#470428}
Committed:
https://chromium.googlesource.com/chromium/src/+/85e629e964e7da6d8a3a05b713e2...
==========

[commit-bot: I haz the power, 2017-05-09 22:12:05]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/85e629e964e7da6d8a3a05b713e2...