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

Issue 713767 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug
Proj-VR
Proj-XR
Proj-XR-VR



Sign in to add a comment

VR Content quad size doesn't match css size

Project Member Reported by amp@chromium.org, Apr 20 2017

Issue description

The content quad is sized for 16:9, but the css size is 960 x 640 resulted in skewed graphcis.
 

Comment 1 by amp@chromium.org, Apr 20 2017

Owner: amp@chromium.org
Status: Started (was: Untriaged)
Project Member

Comment 2 by bugdroid1@chromium.org, Apr 20 2017

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

commit f434f068dd1368d5b762aaba0798362922c6ab2e
Author: amp <amp@chromium.org>
Date: Thu Apr 20 18:12:30 2017

Fix content quad size and translation

BUG= 713767 

Review-Url: https://codereview.chromium.org/2828873005
Cr-Commit-Position: refs/heads/master@{#466072}

[modify] https://crrev.com/f434f068dd1368d5b762aaba0798362922c6ab2e/chrome/browser/android/vr_shell/ui_scene_manager.cc

Comment 3 by amp@chromium.org, Apr 20 2017

Labels: Merge-Request-59
Project Member

Comment 4 by sheriffbot@chromium.org, Apr 20 2017

Labels: -Merge-Request-59 Hotlist-Merge-Approved Merge-Approved-59
Your change meets the bar and is auto-approved for M59. Please go ahead and merge the CL to branch 3071 manually. Please contact milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), gkihumba@(ChromeOS), Abdul Syed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 20 2017

Labels: -merge-approved-59 merge-merged-3071
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/4f935d04dbb6a3ac954a3670b6e41abc1fad0748

commit 4f935d04dbb6a3ac954a3670b6e41abc1fad0748
Author: amp <amp@chromium.org>
Date: Thu Apr 20 23:30:25 2017

Fix content quad size and translation

BUG= 713767 
NOTRY=true
NOPRESUBMIT=true

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

Review-Url: https://codereview.chromium.org/2837443002
Cr-Commit-Position: refs/branch-heads/3071@{#105}
Cr-Branched-From: a106f0abbf69dad349d4aaf4bcc4f5d376dd2377-refs/heads/master@{#464641}

[modify] https://crrev.com/4f935d04dbb6a3ac954a3670b6e41abc1fad0748/chrome/browser/android/vr_shell/ui_scene_manager.cc

Comment 6 by amp@chromium.org, Apr 20 2017

Status: Fixed (was: Started)
Labels: Proj-VR-Shell

Sign in to add a comment