[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

These benchmarks should not be running on non-mobile platforms.

BUG=636790, 630854
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Committed: https://crrev.com/0cdeb2d5f4785e9ccbaf4b6f0235021b0a436cf2
Cr-Commit-Position: refs/heads/master@{#411891}

[perezju, 2016-08-12 15:56:57]

Description was changed from

==========
[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

These benchmarks should not be running on non-mobile platforms.

BUG=636790,630854
==========

to

==========
[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

These benchmarks should not be running on non-mobile platforms.

BUG=636790,630854
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

[perezju, 2016-08-12 15:58:21]

perezju@chromium.org changed reviewers:
+ nednguyen@google.com, rmcilroy@chromium.org

[perezju, 2016-08-12 15:58:21]

If all looks good please feel free to land. (I'll be on a flight most of
Monday.)

[perezju, 2016-08-12 15:58:27]

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

[commit-bot: I haz the power, 2016-08-12 15:58:56]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[rmcilroy, 2016-08-12 16:08:51]

We were using this for some coverage of Ignition on desktop, but we will move to
the V8_browse tests to get that, so this LGTM while we still have this.

[perezju, 2016-08-12 16:16:49]

On 2016/08/12 16:08:51, rmcilroy wrote:
> We were using this for some coverage of Ignition on desktop, but we will move
to
> the V8_browse tests to get that, so this LGTM while we still have this.

Just to clarify, that means to *not* land this until you update the V8_browse
tests?

[rmcilroy, 2016-08-12 16:19:54]

On 2016/08/12 16:16:49, perezju wrote:
> On 2016/08/12 16:08:51, rmcilroy wrote:
> > We were using this for some coverage of Ignition on desktop, but we will
move
> to
> > the V8_browse tests to get that, so this LGTM while we still have this.
> 
> Just to clarify, that means to *not* land this until you update the V8_browse
> tests?

No, I'm fine with landing this now given then desktop bots are having trouble
with the benchmarks. Sorry for the confusion.

[commit-bot: I haz the power, 2016-08-12 17:59:39]

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

[commit-bot: I haz the power, 2016-08-12 17:59:40]

Dry run: Try jobs failed on following builders:
  android_s5_perf_cq on master.tryserver.chromium.perf (JOB_TIMED_OUT, no build
URL)

[nednguyen, 2016-08-12 20:29:41]

lgtm

[perezju, 2016-08-13 10:14:58]

The CQ bit was checked by perezju@chromium.org

[commit-bot: I haz the power, 2016-08-13 10:15:02]

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, 2016-08-13 10:18:13]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-13 10:19:36]

Description was changed from

==========
[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

These benchmarks should not be running on non-mobile platforms.

BUG=636790,630854
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq
==========

to

==========
[tools/perf] Enable v8 top_10_mobile benchmarks on mobile only

These benchmarks should not be running on non-mobile platforms.

BUG=636790,630854
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.perf:android_s5_perf_cq;master.tryserver.chromium.perf:linux_perf_cq;master.tryserver.chromium.perf:mac_retina_perf_cq;master.tryserver.chromium.perf:winx64_10_perf_cq

Committed: https://crrev.com/0cdeb2d5f4785e9ccbaf4b6f0235021b0a436cf2
Cr-Commit-Position: refs/heads/master@{#411891}
==========

[commit-bot: I haz the power, 2016-08-13 10:19:37]

Patchset 1 (id:??) landed as
https://crrev.com/0cdeb2d5f4785e9ccbaf4b6f0235021b0a436cf2
Cr-Commit-Position: refs/heads/master@{#411891}