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

Unified Diff: content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java

Issue 2502763003: Introduce ViewRoot to forward input/view events to native (Closed)
Patch Set: prevent view_root_ gc'ed Created 4 years 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: content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
diff --git a/content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java b/content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
index 77211351fab7e2da8043bee0dc454cdc914814e5..80b3593f3b1d798dce4770b6e686a07776b7ebf5 100644
--- a/content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
+++ b/content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java
@@ -15,6 +15,7 @@ import android.widget.FrameLayout;
import org.chromium.base.annotations.CalledByNative;
import org.chromium.base.annotations.JNINamespace;
import org.chromium.content_public.browser.WebContents;
+import org.chromium.ui.base.ViewRoot;
import org.chromium.ui.base.WindowAndroid;
/***
@@ -71,8 +72,7 @@ public class ContentViewRenderView extends FrameLayout {
nativeSurfaceChanged(mNativeContentViewRenderView,
format, width, height, holder.getSurface());
if (mContentViewCore != null) {
- mContentViewCore.onPhysicalBackingSizeChanged(
- width, height);
+ getViewRoot().onPhysicalBackingSizeChanged(width, height);
}
}
@@ -94,6 +94,10 @@ public class ContentViewRenderView extends FrameLayout {
mSurfaceView.setVisibility(VISIBLE);
}
+ private ViewRoot getViewRoot() {
+ return mContentViewCore.getWindowAndroid().getViewRoot();
+ }
+
/**
* Sets the background color of the surface view. This method is necessary because the
* background color of ContentViewRenderView itself is covered by the background of
@@ -128,7 +132,7 @@ public class ContentViewRenderView extends FrameLayout {
mContentViewCore = contentViewCore;
if (mContentViewCore != null) {
- mContentViewCore.onPhysicalBackingSizeChanged(getWidth(), getHeight());
+ getViewRoot().onPhysicalBackingSizeChanged(getWidth(), getHeight());
nativeSetCurrentWebContents(
mNativeContentViewRenderView, mContentViewCore.getWebContents());
} else {
« no previous file with comments | « content/public/android/java/src/org/chromium/content/browser/ContentViewCore.java ('k') | ui/android/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698