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

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

Issue 201583004: Add layerTreeBuildHelper to control layer tree in ContentViewRenderView (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Git add the new files Created 6 years, 8 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 35ce1c6282427f64444437b06c03c9856167b257..dac3455be09a22beb3bbf7bd46ca313fa7276696 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
@@ -43,7 +43,7 @@ public class ContentViewRenderView extends FrameLayout {
private int mPendingSwapBuffers;
private boolean mNeedToRender;
- private ContentView mCurrentContentView;
+ protected ContentView mCurrentContentView;
Yaron 2014/04/23 20:57:46 Is this change still necessary? You haven't re-upl
Yusuf 2014/04/24 00:06:03 Yes. I will need this in extending classes to rout
private final Runnable mRenderRunnable = new Runnable() {
@Override
@@ -266,6 +266,13 @@ public class ContentViewRenderView extends FrameLayout {
nativeSetOverlayVideoMode(mNativeContentViewRenderView, enabled);
}
+ /**
+ *
Yaron 2014/04/23 20:57:46 Update
Yusuf 2014/04/24 00:06:03 Done.
+ */
+ public void setLayerTreeBuildHelper(long layerTreeBuildHelperNativePtr) {
+ nativeSetLayerTreeBuildHelper(mNativeContentViewRenderView, layerTreeBuildHelperNativePtr);
+ }
+
@CalledByNative
private void requestRender() {
ContentViewCore contentViewCore = mCurrentContentView != null ?
@@ -305,7 +312,7 @@ public class ContentViewRenderView extends FrameLayout {
if (mPendingSwapBuffers > 0) mPendingSwapBuffers--;
}
- private void render() {
+ protected void render() {
if (mPendingRenders > 0) mPendingRenders--;
// Waiting for the content view contents to be ready avoids compositing
@@ -334,6 +341,8 @@ public class ContentViewRenderView extends FrameLayout {
private native void nativeDestroy(long nativeContentViewRenderView);
private native void nativeSetCurrentContentView(long nativeContentViewRenderView,
long nativeContentView);
+ private native void nativeSetLayerTreeBuildHelper(long nativeContentViewRenderView,
+ long buildHelperNativePtr);
private native void nativeSurfaceCreated(long nativeContentViewRenderView);
private native void nativeSurfaceDestroyed(long nativeContentViewRenderView);
private native void nativeSurfaceChanged(long nativeContentViewRenderView,

Powered by Google App Engine
This is Rietveld 408576698