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

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

Issue 2626723003: Revert "Introduce ViewRoot forwarding input/view events to native" (Closed)
Patch Set: 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: 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 80b3593f3b1d798dce4770b6e686a07776b7ebf5..77211351fab7e2da8043bee0dc454cdc914814e5 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,7 +15,6 @@ 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;
/***
@@ -72,7 +71,8 @@ public class ContentViewRenderView extends FrameLayout {
nativeSurfaceChanged(mNativeContentViewRenderView,
format, width, height, holder.getSurface());
if (mContentViewCore != null) {
- getViewRoot().onPhysicalBackingSizeChanged(width, height);
+ mContentViewCore.onPhysicalBackingSizeChanged(
+ width, height);
}
}
@@ -94,10 +94,6 @@ 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
@@ -132,7 +128,7 @@ public class ContentViewRenderView extends FrameLayout {
mContentViewCore = contentViewCore;
if (mContentViewCore != null) {
- getViewRoot().onPhysicalBackingSizeChanged(getWidth(), getHeight());
+ mContentViewCore.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