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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java

Issue 2635123002: Fix VrShell resizing incorrectly by setting the CompositorViewHolder to a fixed size when in VR. (Closed)
Patch Set: Set CompositorViewHolder size instead Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java
index 067749b5ba56c11a87ff12b8b1a22f3270a552b9..7c4750deab9353c081108d20e52ffb2a4b81b665 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrClassesBuilderImpl.java
@@ -9,6 +9,7 @@ import android.os.StrictMode;
import org.chromium.base.Log;
import org.chromium.base.annotations.UsedByReflection;
+import org.chromium.chrome.browser.compositor.CompositorViewHolder;
/**
* Builder class to create all VR related classes. These VR classes are behind the same build time
@@ -38,10 +39,10 @@ public class VrClassesBuilderImpl implements VrClassesBuilder {
}
@Override
- public VrShell createVrShell() {
+ public VrShell createVrShell(CompositorViewHolder compositorViewHolder) {
StrictMode.ThreadPolicy oldPolicy = StrictMode.allowThreadDiskReads();
try {
- return new VrShellImpl(mActivity);
+ return new VrShellImpl(mActivity, compositorViewHolder);
} catch (Exception ex) {
Log.e(TAG, "Unable to instantiate VrShellImpl", ex);
return null;

Powered by Google App Engine
This is Rietveld 408576698