Relocate stability report extraction tests to seperate file

Relocate stability report extraction tests to seperate file

BUG=691595
Review-Url: https://codereview.chromium.org/2926113002
Cr-Commit-Position: refs/heads/master@{#477725}
Committed: https://chromium.googlesource.com/chromium/src/+/1a95d3f14a92b9377d8d94885414a6b1ce2710db

[manzagop (departed), 2017-06-07 14:52:44]

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

[commit-bot: I haz the power, 2017-06-07 14:53:06]

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-06-07 14:59:02]

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

[commit-bot: I haz the power, 2017-06-07 14:59:16]

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-06-07 15:00:29]

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

[manzagop (departed), 2017-06-07 15:00:29]

Hi Brian,

Relocating some code: nothing new! Could you have a look?

Thanks!

P-A

[bcwhite, 2017-06-07 16:06:35]

On 2017/06/07 15:00:29, manzagop wrote:
> Hi Brian,
> 
> Relocating some code: nothing new! Could you have a look?
> 
> Thanks!
> 
> P-A

I see a lot of code deleted but very little added.  Is there supposed to be
another file?

[manzagop (departed), 2017-06-07 16:08:32]

On 2017/06/07 16:06:35, bcwhite wrote:
> On 2017/06/07 15:00:29, manzagop wrote:
> > Hi Brian,
> > 
> > Relocating some code: nothing new! Could you have a look?
> > 
> > Thanks!
> > 
> > P-A
> 
> I see a lot of code deleted but very little added.  Is there supposed to be
> another file?

The new file (stability_report_extractor_unittest.cc) is there but the tool 
accurately picked up it's branched. So yeah, the true diff is super small!

[commit-bot: I haz the power, 2017-06-07 17:22:41]

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

[commit-bot: I haz the power, 2017-06-07 17:22:42]

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

[bcwhite, 2017-06-07 17:23:50]

lgtm

[manzagop (departed), 2017-06-07 17:49:15]

Thanks for the review!

[manzagop (departed), 2017-06-07 17:49:33]

The CQ bit was checked by manzagop@chromium.org

[manzagop (departed), 2017-06-07 17:49:34]

The patchset sent to the CQ was uploaded after l-g-t-m from bcwhite@chromium.org
Link to the patchset: https://codereview.chromium.org/2926113002/#ps40001
(title: "remove unused var/function (thanks clang)")

[commit-bot: I haz the power, 2017-06-07 17:49: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-06-07 19:29:08]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1496857773876210,
"parent_rev": "af60565e44dd99ae730253242f2b06a659d577ad", "commit_rev":
"1a95d3f14a92b9377d8d94885414a6b1ce2710db"}

[commit-bot: I haz the power, 2017-06-07 19:29:21]

Description was changed from

==========
Relocate stability report extraction tests to seperate file

BUG=691595
==========

to

==========
Relocate stability report extraction tests to seperate file

BUG=691595

Review-Url: https://codereview.chromium.org/2926113002
Cr-Commit-Position: refs/heads/master@{#477725}
Committed:
https://chromium.googlesource.com/chromium/src/+/1a95d3f14a92b9377d8d94885414...
==========

[commit-bot: I haz the power, 2017-06-07 19:29:22]

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