Convert octane test score to float

Convert octane test score to float first
to prevent fails on slow devices

BUG=508907
CQ_EXTRA_TRYBOTS=tryserver.chromium.perf:linux_perf_bisect;tryserver.chromium.perf:mac_perf_bisect;tryserver.chromium.perf:win_perf_bisect

Committed: https://crrev.com/3ac26e57adcdb0b08006a5f5f1e69e3615a28432
Cr-Commit-Position: refs/heads/master@{#342772}

[alexanderk, 2015-07-23 11:24:50]

alexanderk@opera.com changed reviewers:
+ dtu@chromium.org, nednguyen@google.com

[nednguyen, 2015-07-23 14:59:01]

https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
File tools/perf/benchmarks/octane.py (right):

https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
tools/perf/benchmarks/octane.py:119: score = int(float(score_and_name[1]))
why not just float(...)? I don't think we need to cast this to int.

[nednguyen, 2015-07-23 14:59:33]

On 2015/07/23 14:59:01, nednguyen wrote:
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> File tools/perf/benchmarks/octane.py (right):
> 
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> tools/perf/benchmarks/octane.py:119: score = int(float(score_and_name[1]))
> why not just float(...)? I don't think we need to cast this to int.

lgtm btw

[alexanderk, 2015-07-24 11:53:03]

On 2015/07/23 14:59:33, nednguyen wrote:
> On 2015/07/23 14:59:01, nednguyen wrote:
> >
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> > File tools/perf/benchmarks/octane.py (right):
> > 
> >
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> > tools/perf/benchmarks/octane.py:119: score = int(float(score_and_name[1]))
> > why not just float(...)? I don't think we need to cast this to int.
> 
> lgtm btw

Right, just float should be OK

[nednguyen, 2015-07-24 13:07:22]

On 2015/07/24 11:53:03, alexanderk wrote:
> On 2015/07/23 14:59:33, nednguyen wrote:
> > On 2015/07/23 14:59:01, nednguyen wrote:
> > >
> >
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> > > File tools/perf/benchmarks/octane.py (right):
> > > 
> > >
> >
>
https://codereview.chromium.org/1230063007/diff/1/tools/perf/benchmarks/octan...
> > > tools/perf/benchmarks/octane.py:119: score = int(float(score_and_name[1]))
> > > why not just float(...)? I don't think we need to cast this to int.
> > 
> > lgtm btw
> 
> Right, just float should be OK

Lgtm

[dtu, 2015-07-24 20:06:17]

lgtm

Thanks for the fix!

[alexanderk, 2015-07-27 07:40:23]

On 2015/07/24 20:06:17, dtu wrote:
> lgtm
> 
> Thanks for the fix!

np

[dtu, 2015-08-10 23:24:13]

The CQ bit was checked by dtu@chromium.org

[commit-bot: I haz the power, 2015-08-10 23:25:11]

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

[commit-bot: I haz the power, 2015-08-11 01:27:14]

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

[commit-bot: I haz the power, 2015-08-11 01:27:17]

Try jobs failed on following builders:
  android_nexus5_perf_bisect on tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)

[nednguyen, 2015-08-11 01:36:11]

The CQ bit was checked by nednguyen@google.com

[commit-bot: I haz the power, 2015-08-11 01:36:45]

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

[commit-bot: I haz the power, 2015-08-11 01:42:44]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-08-11 01:43:43]

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