Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(507)

Issue 2724643002: Reduce number of VR test APKs (Closed)

Created:
3 years, 9 months ago by bsheedy
Modified:
3 years, 9 months ago
Reviewers:
bajones
CC:
chromium-reviews, feature-vr-reviews_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Reduce number of VR test APKs Removes some older versions of the test APKs, but adds a hash history file in case any need to be manually downloaded. With the addition of the VR Services version checker, we should only need to run automated tests with the current release version (for functionality testing) and with a version that is considered outdated (to test that the prompt is displayed and VR functionality is disabled). Since don't really interact with Daydream Home (at least currently), we can just keep the most recent version around for testing. BUG= Review-Url: https://codereview.chromium.org/2724643002 Cr-Commit-Position: refs/heads/master@{#453960} Committed: https://chromium.googlesource.com/chromium/src/+/d6eee8d17b1a09b11bfaa6b72aee5697aa5520e7

Patch Set 1 #

Messages

Total messages: 8 (4 generated)
bsheedy
PTAL
3 years, 9 months ago (2017-02-28 20:14:49 UTC) #2
bajones
On 2017/02/28 20:14:49, bsheedy wrote: > PTAL LGTM
3 years, 9 months ago (2017-03-01 06:47:59 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2724643002/1
3 years, 9 months ago (2017-03-01 17:27:18 UTC) #5
commit-bot: I haz the power
3 years, 9 months ago (2017-03-01 17:37:38 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/d6eee8d17b1a09b11bfaa6b72aee...

Powered by Google App Engine
This is Rietveld 408576698