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

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

Issue 2727873002: Implement lazy initialization for VrShellDelegate (Closed)
Patch Set: Fix support for DON flow from headset insertion. Created 3 years, 10 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/VrShellImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
index 20b26f4d5eb4b31195b8858f423f10154f3b9560..549aee1a11bff353a97b7b2603f8ed84bd74aac0 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/vr_shell/VrShellImpl.java
@@ -28,7 +28,6 @@ import org.chromium.chrome.browser.ChromeSwitches;
import org.chromium.chrome.browser.ChromeVersionInfo;
import org.chromium.chrome.browser.NativePage;
import org.chromium.chrome.browser.WebContentsFactory;
-import org.chromium.chrome.browser.compositor.CompositorViewHolder;
import org.chromium.chrome.browser.omnibox.geo.GeolocationHeader;
import org.chromium.chrome.browser.tab.EmptyTabObserver;
import org.chromium.chrome.browser.tab.Tab;
@@ -74,7 +73,6 @@ public class VrShellImpl extends GvrLayout implements VrShell, SurfaceHolder.Cal
private final ChromeActivity mActivity;
private final VrShellDelegate mDelegate;
- private final CompositorViewHolder mCompositorViewHolder;
private final VirtualDisplayAndroid mContentVirtualDisplay;
private final VirtualDisplayAndroid mUiVirtualDisplay;
private final TabRedirectHandler mTabRedirectHandler;
@@ -109,13 +107,12 @@ public class VrShellImpl extends GvrLayout implements VrShell, SurfaceHolder.Cal
private float mLastContentHeight;
private float mLastContentDpr;
- public VrShellImpl(ChromeActivity activity, VrShellDelegate delegate,
- CompositorViewHolder compositorViewHolder) {
+ public VrShellImpl(
+ ChromeActivity activity, VrShellDelegate delegate, TabModelSelector tabModelSelector) {
super(activity);
mActivity = activity;
mDelegate = delegate;
- mCompositorViewHolder = compositorViewHolder;
- mTabModelSelector = mCompositorViewHolder.detachForVR();
+ mTabModelSelector = tabModelSelector;
mUiCVCContainer = new FrameLayout(getContext()) {
@Override
public boolean dispatchTouchEvent(MotionEvent event) {
@@ -461,7 +458,6 @@ public class VrShellImpl extends GvrLayout implements VrShell, SurfaceHolder.Cal
mContentVirtualDisplay.destroy();
mUiVirtualDisplay.destroy();
super.shutdown();
- mCompositorViewHolder.onExitVR(mTabModelSelector);
}
@Override

Powered by Google App Engine
This is Rietveld 408576698