New issue
Advanced search Search tips

Issue 779837 link

Starred by 2 users

Issue metadata

Status: WontFix
Owner:
Closed: Jun 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-XR

Blocked on:
issue 771171



Sign in to add a comment

Support VR Telemetry tests on Raspberry Pis

Project Member Reported by bsheedy@chromium.org, Oct 30 2017

Issue description

We've managed to hack Telemetry (or more specifically, Devil) to work on a Raspberry Pi. However, there are a couple of issues currently:

1. This was done using arm dependencies obtained from places like peoples' personal github repos. In order to officially add arm support to Devil, we'll need to build the necessary dependencies ourselves.

2. The tests take way too long, largely due to the extra heat that's generated by unnecessary rendering while the trace data is being processed (takes ~45 seconds on a Raspberry Pi, so that's a lot of heat).
 
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 31 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/82a47597e79c96751e2cdd3ba3915018068aab7b

commit 82a47597e79c96751e2cdd3ba3915018068aab7b
Author: bsheedy <bsheedy@chromium.org>
Date: Tue Oct 31 22:29:01 2017

Make VR Telemetry tests navigate to blank page on finish

Re-lands a prior patch that made VR Telemetry tests exit VR and navigate
to a blank page after each story to reduce heat output, with a few
differences:

1. The issue that caused the original patch's revert has been fixed by
    cycling the device's screen off then on instead of just ensuring
    that it is on.
2. Instead of running JavaScript code to exit VR then navigate, we
    simply navigate away since there doesn't seem to be any impact to
    stability or results by navigating without first exiting VR.

Bug:  779837 
Change-Id: Ia030b52d2cb87026f073eda2e7d37209c0992dae
Reviewed-on: https://chromium-review.googlesource.com/747278
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#512980}
[modify] https://crrev.com/82a47597e79c96751e2cdd3ba3915018068aab7b/tools/perf/contrib/vr_benchmarks/shared_android_vr_page_state.py
[modify] https://crrev.com/82a47597e79c96751e2cdd3ba3915018068aab7b/tools/perf/contrib/vr_benchmarks/webvr_sample_pages.py

Blockedon: 771171
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 20 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/e29eea14f91a8adb8ed3a443fb23eab673f2932b

commit e29eea14f91a8adb8ed3a443fb23eab673f2932b
Author: bsheedy <bsheedy@chromium.org>
Date: Sat Jan 20 01:14:09 2018

Add build targets for Devil arm deps

Adds build targets for arm versions of md5sum, host_forwarder, and
libc++. Special targets are necessary since the Android/device deps
can't be built with the clang_arm toolchain, so we can't just use it
for everything.

Also updates the arm build config slightly since Raspberry Pis require
mfloat-abi to be set to hard, while Android requires it to be set to
soft.

Bug:  779837 
Change-Id: I19c861d2c8a528c82bcb5d9f43c3812499d23ddf
Reviewed-on: https://chromium-review.googlesource.com/761357
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#530711}
[modify] https://crrev.com/e29eea14f91a8adb8ed3a443fb23eab673f2932b/build/config/arm.gni
[modify] https://crrev.com/e29eea14f91a8adb8ed3a443fb23eab673f2932b/build/config/gcc/BUILD.gn
[modify] https://crrev.com/e29eea14f91a8adb8ed3a443fb23eab673f2932b/build/secondary/third_party/catapult/devil/BUILD.gn
[add] https://crrev.com/e29eea14f91a8adb8ed3a443fb23eab673f2932b/build/secondary/third_party/catapult/devil/devil_arm.gni

Status: WontFix (was: Started)
We've decided to use Intel NUCs instead of Raspberry Pis, since they're x86 and still relatively cheap, so we don't have to deal with arm issues. They're also significantly faster than the RPis.
Components: Internals>XR

Sign in to add a comment