commit fc6affc60d2298600c1c8433fd918226f4f38a5b
author magjed <magjed@webrtc.org> Thu Nov 12 01:08:32 2015 -0800
committer Commit bot <commit-bot@chromium.org> Thu Nov 12 09:08:37 2015 +0000
tree 2ee289265bc91d806a8a0f3041bbcf048ea0790d
parent 96839648a0eb09476f07791fdf3ff2facffa9a8a

Android SurfaceViewRenderer: Call glClear() for every frame to avoid bad GL state

BUG=webrtc:5147

Review URL: https://codereview.webrtc.org/1436883002

Cr-Commit-Position: refs/heads/master@{#10617}

talk/app/webrtc/java/android/org/webrtc/SurfaceViewRenderer.java

diff --git a/talk/app/webrtc/java/android/org/webrtc/SurfaceViewRenderer.java b/talk/app/webrtc/java/android/org/webrtc/SurfaceViewRenderer.java
index c999947..c4785c9 100644
--- a/talk/app/webrtc/java/android/org/webrtc/SurfaceViewRenderer.java
+++ b/talk/app/webrtc/java/android/org/webrtc/SurfaceViewRenderer.java
@@ -465,6 +465,9 @@
     }
 
     GLES20.glViewport(0, 0, surfaceWidth, surfaceHeight);
+    // TODO(magjed): glClear() shouldn't be necessary since every pixel is covered anyway, but it's
+    // a workaround for bug 5147. Performance will be slightly worse.
+    GLES20.glClear(GLES20.GL_COLOR_BUFFER_BIT);
     if (frame.yuvFrame) {
       // Make sure YUV textures are allocated.
       if (yuvTextures == null) {