Rejig stability intstrumentation filenames and targets

Rejig stability intstrumentation filenames and targets

The main change is to rename postmortem_report_extractor to stability_report_extractor, and pull it out to a separate target. We want to use this not only postmortem, but also on crash.

BUG=620813
Review-Url: https://codereview.chromium.org/2856923002
Cr-Commit-Position: refs/heads/master@{#469438}
Committed: https://chromium.googlesource.com/chromium/src/+/309a96117edbe53789c764bfcff76c13b0e88b52

[manzagop (departed), 2017-05-02 16:08:21]

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

[commit-bot: I haz the power, 2017-05-02 16:08:38]

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-02 16:12:58]

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

[commit-bot: I haz the power, 2017-05-02 16:12:59]

Dry run: Try jobs failed on following builders:
  android_arm64_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_arm6...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_tsan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[manzagop (departed), 2017-05-02 17:40:40]

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

[manzagop (departed), 2017-05-02 17:40:47]

Patchset #1 (id:1) has been deleted

[manzagop (departed), 2017-05-02 17:40:54]

Patchset #1 (id:20001) has been deleted

[commit-bot: I haz the power, 2017-05-02 17:40:55]

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-02 17:42:45]

Description was changed from

==========
Rejig stability intstrumentation filenames and targets

BUG=620813
==========

to

==========
Rejig stability intstrumentation filenames and targets

The main change is to rename postmortem_report_extractor to
stability_report_extractor, and pull it out to a separate target. We want to use
this not only postmortem, but also on crash.

BUG=620813
==========

[commit-bot: I haz the power, 2017-05-02 17:44:59]

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

[commit-bot: I haz the power, 2017-05-02 17:45:00]

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

[manzagop (departed), 2017-05-02 17:55:36]

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

[commit-bot: I haz the power, 2017-05-02 17:55:55]

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-02 19:03:31]

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

[commit-bot: I haz the power, 2017-05-02 19:03:31]

Dry run: This issue passed the CQ dry run.

[manzagop (departed), 2017-05-02 19:33:19]

Patchset #1 (id:40001) has been deleted

[manzagop (departed), 2017-05-02 19:37:59]

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

[manzagop (departed), 2017-05-02 19:37:59]

Hi Brian,
This moves a few things around, nothing added. Could you have a look?
Thanks!
Pierre

[manzagop (departed), 2017-05-03 18:46:09]

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

[commit-bot: I haz the power, 2017-05-03 18:47:23]

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-03 19:36:28]

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

[commit-bot: I haz the power, 2017-05-03 19:36:28]

Dry run: Try jobs failed on following builders:
  win_chromium_compile_dbg_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_comp...)

[manzagop (departed), 2017-05-03 20:41:21]

Patchset #2 (id:80001) has been deleted

[manzagop (departed), 2017-05-03 21:41:04]

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

[commit-bot: I haz the power, 2017-05-03 21:42:04]

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-03 21:46:06]

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

[manzagop (departed), 2017-05-03 21:46:06]

Hi Scott,

Could you have a look at this? It's renames a file and rejigs some targets, but
there's no actual change.

It's setup for the Crashpad integration CL which I'll send to Mark. I think
Brian is OOO.

Thanks,

Pierre

[scottmg, 2017-05-03 21:56:11]

lgtm

[manzagop (departed), 2017-05-03 21:58:28]

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

[manzagop (departed), 2017-05-03 21:58:29]

Thanks Scott!

[manzagop (departed), 2017-05-03 21:59:25]

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

[manzagop (departed), 2017-05-03 21:59:26]

Hi Rob,

Could you have an OWNERS' look at:
    chrome\browser\chrome_browser_field_trials_desktop.cc
    chrome\browser\metrics\chrome_metrics_service_client.cc
    components\metrics\BUILD.gn

Thanks!

P-A

[commit-bot: I haz the power, 2017-05-03 22:16:46]

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

[commit-bot: I haz the power, 2017-05-03 22:16:47]

Dry run: Try jobs failed on following builders:
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[manzagop (departed), 2017-05-04 13:17:10]

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

[commit-bot: I haz the power, 2017-05-04 13:17:21]

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-04 14:21:02]

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

[commit-bot: I haz the power, 2017-05-04 14:21:03]

Dry run: This issue passed the CQ dry run.

[manzagop (departed), 2017-05-04 19:12:56]

Friendly ping. The OWNERS review is pretty small. :)

[rkaplow1, 2017-05-04 19:43:50]

rkaplow@google.com changed reviewers:
+ rkaplow@google.com

[rkaplow1, 2017-05-04 19:43:51]

lgtm

[rkaplow, 2017-05-04 19:44:18]

lgtm

whoops wrong account

[manzagop (departed), 2017-05-04 19:48:19]

Thanks!

[manzagop (departed), 2017-05-04 19:48:25]

The CQ bit was checked by manzagop@chromium.org

[manzagop (departed), 2017-05-04 19:48:25]

The patchset sent to the CQ was uploaded after l-g-t-m from scottmg@chromium.org
Link to the patchset: https://codereview.chromium.org/2856923002/#ps120001
(title: "Missing include")

[commit-bot: I haz the power, 2017-05-04 19:49:08]

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-04 19:55:41]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1493927305277620,
"parent_rev": "41a331f01308223da331ecbdf5d73f4fb03fe42e", "commit_rev":
"309a96117edbe53789c764bfcff76c13b0e88b52"}

[commit-bot: I haz the power, 2017-05-04 19:55:54]

Description was changed from

==========
Rejig stability intstrumentation filenames and targets

The main change is to rename postmortem_report_extractor to
stability_report_extractor, and pull it out to a separate target. We want to use
this not only postmortem, but also on crash.

BUG=620813
==========

to

==========
Rejig stability intstrumentation filenames and targets

The main change is to rename postmortem_report_extractor to
stability_report_extractor, and pull it out to a separate target. We want to use
this not only postmortem, but also on crash.

BUG=620813

Review-Url: https://codereview.chromium.org/2856923002
Cr-Commit-Position: refs/heads/master@{#469438}
Committed:
https://chromium.googlesource.com/chromium/src/+/309a96117edbe53789c764bfcff7...
==========

[commit-bot: I haz the power, 2017-05-04 19:55:55]

Committed patchset #3 (id:120001) as
https://chromium.googlesource.com/chromium/src/+/309a96117edbe53789c764bfcff7...