Commit 27b47d19 authored by Tibor Goldschwendt's avatar Tibor Goldschwendt Committed by Commit Bot

[vr] Zoom in pages in VR by ~20%.

Bug: 737118
Change-Id: Ib14dfb4207a5cf37b173852a6a4fc13c9d13019f
Reviewed-on: https://chromium-review.googlesource.com/575074Reviewed-by: default avatarMichael Thiessen <mthiesse@chromium.org>
Commit-Queue: Tibor Goldschwendt <tiborg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#488402}
parent b542c0ba
......@@ -70,15 +70,16 @@ public class VrShellImpl
// a way to compute good values for any screen size/scaling ratio.
// Increasing DPR any more than this doesn't appear to increase text quality.
private static final float DEFAULT_DPR = 1.4f;
private static final float DEFAULT_DPR = 1.7f;
// Fairly arbitrary values that put a good amount of content on the screen without making the
// text too small to read.
@VisibleForTesting
public static final float DEFAULT_CONTENT_WIDTH = 800f;
public static final float DEFAULT_CONTENT_WIDTH = 645f;
@VisibleForTesting
public static final float DEFAULT_CONTENT_HEIGHT = 533f;
public static final float DEFAULT_CONTENT_HEIGHT = 430f;
// Make full screen 16:9 until we get exact dimensions from playing video.
private static final float FULLSCREEN_DPR = 1.4f;
private static final float FULLSCREEN_CONTENT_WIDTH = 1024f;
private static final float FULLSCREEN_CONTENT_HEIGHT = 576f;
......@@ -493,7 +494,7 @@ public class VrShellImpl
@CalledByNative
public void onFullscreenChanged(boolean enabled) {
if (enabled) {
setContentCssSize(FULLSCREEN_CONTENT_WIDTH, FULLSCREEN_CONTENT_HEIGHT, DEFAULT_DPR);
setContentCssSize(FULLSCREEN_CONTENT_WIDTH, FULLSCREEN_CONTENT_HEIGHT, FULLSCREEN_DPR);
} else {
setContentCssSize(DEFAULT_CONTENT_WIDTH, DEFAULT_CONTENT_HEIGHT, DEFAULT_DPR);
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment