diff options
author | jinho.bang@samsung.com <jinho.bang@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-28 23:46:37 +0000 |
---|---|---|
committer | jinho.bang@samsung.com <jinho.bang@samsung.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-28 23:46:37 +0000 |
commit | d9778cca7adc98309edcaff49f323b9490af9fb9 (patch) | |
tree | d210c76d30feac86d69c29e145a0d3052197a8ca | |
parent | 5011b1074b90b83588f1ad2d5d3fc122ed0726c8 (diff) | |
download | chromium_src-d9778cca7adc98309edcaff49f323b9490af9fb9.zip chromium_src-d9778cca7adc98309edcaff49f323b9490af9fb9.tar.gz chromium_src-d9778cca7adc98309edcaff49f323b9490af9fb9.tar.bz2 |
Returning back from any app [after share via] is creating a black screen
When the surfaceView is destroyed during compositing, ContentViewRenderView
can't receive the message. So, the onSwapBuffers event will not occurs, and the
value of mPendingSwapBuffers will increasing without bound.
To prevent the problem, currently, the value is initialized in onReadyToRender
method. However, content_shell and chromium_test_shell override the method and
don't call super method.
BUG=337770
Review URL: https://codereview.chromium.org/134353004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@247525 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r-- | content/public/android/java/src/org/chromium/content/browser/ContentViewRenderView.java | 6 |
1 files changed, 4 insertions, 2 deletions
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 6b7706b..90481f6 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 @@ -78,6 +78,10 @@ public class ContentViewRenderView extends FrameLayout { public void surfaceCreated(SurfaceHolder holder) { assert mNativeContentViewRenderView != 0; nativeSurfaceCreated(mNativeContentViewRenderView, holder.getSurface()); + + mPendingSwapBuffers = 0; + mPendingRenders = 0; + onReadyToRender(); } @@ -207,8 +211,6 @@ public class ContentViewRenderView extends FrameLayout { * render. */ protected void onReadyToRender() { - mPendingSwapBuffers = 0; - mPendingRenders = 0; } /** |