Enabled VR Shell unittests on main waterfall

Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964
Committed: https://crrev.com/5750756522296b2a9a08009d8d2cc90db3b88f56
Cr-Commit-Position: refs/heads/master@{#434843}

[bsheedy, 2016-11-23 00:45:31]

Description was changed from

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=
==========

to

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964
==========

[bsheedy, 2016-11-23 00:48:30]

bsheedy@chromium.org changed reviewers:
+ bshe@chromium.org, stip@chromium.org

[bsheedy, 2016-11-23 00:48:30]

bshe@chromium.org: Please see change to BUILD file

stip@chromium.org: Please see changes to .json file

[bshe, 2016-11-23 14:27:52]

On 2016/11/23 00:48:30, bsheedy wrote:
> mailto:bshe@chromium.org: Please see change to BUILD file
> 
> mailto:stip@chromium.org: Please see changes to .json file

lgtm

[ghost stip (do not use), 2016-11-28 21:56:22]

stip@chromium.org changed reviewers:
+ jbudorick@chromium.org

[ghost stip (do not use), 2016-11-28 21:56:28]

https://codereview.chromium.org/2521883003/diff/1/testing/buildbot/chromium.a...
File testing/buildbot/chromium.android.json (right):

https://codereview.chromium.org/2521883003/diff/1/testing/buildbot/chromium.a...
testing/buildbot/chromium.android.json:2498: "device_os": "KTU84P",
we have a bit of a capacity crunch on the N5s right now. can you estimate how
many minutes of device time this test will take?

[bsheedy, 2016-11-28 21:58:54]

On 2016/11/28 21:56:28, stip wrote:
>
https://codereview.chromium.org/2521883003/diff/1/testing/buildbot/chromium.a...
> File testing/buildbot/chromium.android.json (right):
> 
>
https://codereview.chromium.org/2521883003/diff/1/testing/buildbot/chromium.a...
> testing/buildbot/chromium.android.json:2498: "device_os": "KTU84P",
> we have a bit of a capacity crunch on the N5s right now. can you estimate how
> many minutes of device time this test will take?

Currently, the tests themselves are taking only a few milliseconds to run, and
the entire test step is taking 12 seconds (see
https://build.chromium.org/p/chromium.fyi/builders/Android%20VR%20Tests/build...).

[ghost stip (do not use), 2016-11-29 00:56:33]

lgtm

[bsheedy, 2016-11-29 00:57:06]

The CQ bit was checked by bsheedy@chromium.org

[commit-bot: I haz the power, 2016-11-29 00:58: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, 2016-11-29 02:48:28]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1480381026751690, "parent_rev":
"2c4ec16ae201ad4d7fbee4ab7fc7bea9134321dc", "commit_rev":
"3d407f87ae2320024b3c9ae93fde970529731500"}

[commit-bot: I haz the power, 2016-11-29 02:48:53]

Description was changed from

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964
==========

to

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964
==========

[commit-bot: I haz the power, 2016-11-29 02:48:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-11-29 02:53:34]

Description was changed from

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964
==========

to

==========
Enabled VR Shell unittests on main waterfall

Changes the VR Shell unittests to compile when WebVR is enabled and enables
them on chromium.android builders.

BUG=667964

Committed: https://crrev.com/5750756522296b2a9a08009d8d2cc90db3b88f56
Cr-Commit-Position: refs/heads/master@{#434843}
==========

[commit-bot: I haz the power, 2016-11-29 02:53:34]

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