Add commit position to bisect result output.

Add commit position to bisect result output.

BUG=413772
NOTRY=true

Committed: https://crrev.com/e0f01f8d8c79983e2cceddfd655f51ea3bccc3a7
Cr-Commit-Position: refs/heads/master@{#296776}

[chrisphan, 2014-09-18 21:29:23]

chrisphan@chromium.org changed reviewers:
+ prasadv@chromium.org

[chrisphan, 2014-09-18 21:29:42]

+reviewers

[prasadv, 2014-09-24 21:48:36]

lgtm

https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression.py
File tools/bisect-perf-regression.py (right):

https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression...
tools/bisect-perf-regression.py:2678: ['Depot', 'Position', 'SHA', 'Mean', 'Std.
Error', 'State'])
Can we use Commit-Position or Position-num

[chrisphan, 2014-09-24 21:56:50]

https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression.py
File tools/bisect-perf-regression.py (right):

https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression...
tools/bisect-perf-regression.py:2678: ['Depot', 'Position', 'SHA', 'Mean', 'Std.
Error', 'State'])
On 2014/09/24 21:48:36, prasadv wrote:
> Can we use Commit-Position or Position-num

I was looking at the result in plain text, it might be easier to read using one
word.  What do you think?

[prasadv, 2014-09-25 16:43:08]

On 2014/09/24 21:56:50, chrisphan wrote:
>
https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression.py
> File tools/bisect-perf-regression.py (right):
> 
>
https://codereview.chromium.org/580293002/diff/1/tools/bisect-perf-regression...
> tools/bisect-perf-regression.py:2678: ['Depot', 'Position', 'SHA', 'Mean',
'Std.
> Error', 'State'])
> On 2014/09/24 21:48:36, prasadv wrote:
> > Can we use Commit-Position or Position-num
> 
> I was looking at the result in plain text, it might be easier to read using
one
> word.  What do you think?

lgtm

[chrisphan, 2014-09-25 18:52:05]

The CQ bit was checked by chrisphan@chromium.org

[commit-bot: I haz the power, 2014-09-25 18:53:21]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/580293002/1

[commit-bot: I haz the power, 2014-09-25 19:03:21]

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

[commit-bot: I haz the power, 2014-09-25 19:03:22]

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

[chrisphan, 2014-09-25 19:24:01]

The CQ bit was checked by chrisphan@chromium.org

[commit-bot: I haz the power, 2014-09-25 19:27:03]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/580293002/20001

[commit-bot: I haz the power, 2014-09-25 19:39:08]

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

[commit-bot: I haz the power, 2014-09-25 19:39:09]

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

[chrisphan, 2014-09-25 20:02:15]

The CQ bit was checked by chrisphan@chromium.org

[commit-bot: I haz the power, 2014-09-25 20:03:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/580293002/20001

[commit-bot: I haz the power, 2014-09-25 20:04:36]

Committed patchset #2 (id:20001) as c5f7b00a53528ec4d7e082bdc41948023e62398e

[commit-bot: I haz the power, 2014-09-25 20:05:21]

Patchset 2 (id:??) landed as
https://crrev.com/e0f01f8d8c79983e2cceddfd655f51ea3bccc3a7
Cr-Commit-Position: refs/heads/master@{#296776}

[fgorski, 2014-09-25 20:33:36]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/608643002/ by fgorski@chromium.org.

The reason for reverting is: Checkperms failure on
https://build.chromium.org/p/chromium.chromiumos/builders/Linux%20ChromiumOS%....