New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 671224 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Dec 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 1
Type: Bug
Proj-XR

Blocking:
issue 682806



Sign in to add a comment

Enable Cardboard support on Daydream device

Project Member Reported by bshe@chromium.org, Dec 5 2016

Issue description

It looks like there is a workaround for Cardboard support for our ChromeTabbedActivity.

We just need to manually check if the current viewer type is Daydream or not and then use different code path to decide if DON is necessary. This workaround restrict to Daydream ready device though.
 

Comment 1 by bshe@chromium.org, Dec 5 2016

Labels: Proj-VR

Comment 3 by bshe@chromium.org, Dec 5 2016

Labels: Merge-Request-56

Comment 4 by dimu@chromium.org, Dec 5 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Components: Blink>WebVR UI>Browser>VR
Project Member

Comment 6 by bugdroid1@chromium.org, Dec 7 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ca4176715d4e0159beec6d08d8420c19e2c7b4f8

commit ca4176715d4e0159beec6d08d8420c19e2c7b4f8
Author: bshe <bshe@chromium.org>
Date: Wed Dec 07 21:17:24 2016

Enable Cardboard support on Daydream ready device

BUG= 671224 

Review-Url: https://codereview.chromium.org/2547973002
Cr-Commit-Position: refs/heads/master@{#436420}
(cherry picked from commit f5e72ebfb73d97c2d7175d2fb578c48006b66c22)

Review URL: https://codereview.chromium.org/2548363009 .

Cr-Commit-Position: refs/branch-heads/2924@{#388}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/ca4176715d4e0159beec6d08d8420c19e2c7b4f8/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellDelegate.java
[modify] https://crrev.com/ca4176715d4e0159beec6d08d8420c19e2c7b4f8/chrome/browser/android/vr_shell/vr_shell_delegate.cc

Comment 7 by bshe@chromium.org, Dec 7 2016

Status: Fixed (was: Started)
Labels: Proj-VR-Cardboard
Components: Internals>VR
Components: -UI>Browser>VR
Blocking: 682806
Components: Internals>XR
Components: Blink>WebXR

Sign in to add a comment