commit 3d8edd862db86174ac0e06b3cce85b49d711ce4e
author Kaiyi Li <kaiyili@google.com> Mon May 10 18:30:38 2021 -0700
committer Kaiyi Li <kaiyili@google.com> Mon May 10 18:33:52 2021 -0700
tree f773103304d3a35e6d251707a42455b1211fabbd
parent 2f3135be375778a660b9e904ee3ced9b079c7fa9

Calling glCheckFramebufferStatus on the guest all the time

Remove all the dependency to glCheckFramebufferStatus_enc, which will
send the call to the host.

Test: Run the emulator. No glCheckFramebufferStatus encode in the guest perfetto trace.
Change-Id: I1a623a0784909dd18aec917c9b7d5b5bc0a8b914

system/GLESv2_enc/GL2Encoder.cpp

diff --git a/system/GLESv2_enc/GL2Encoder.cpp b/system/GLESv2_enc/GL2Encoder.cpp
index db65fb0..4640e70 100755
--- a/system/GLESv2_enc/GL2Encoder.cpp
+++ b/system/GLESv2_enc/GL2Encoder.cpp
@@ -1463,7 +1463,7 @@
 
     if (!has_client_vertex_arrays && !has_indirect_arrays) {
         // ALOGW("glDrawElements: no vertex arrays / buffers bound to the command\n");
-        GLenum status = ctx->m_glCheckFramebufferStatus_enc(self, GL_FRAMEBUFFER);
+        GLenum status = ctx->glCheckFramebufferStatus(self, GL_FRAMEBUFFER);
         SET_ERROR_IF(status != GL_FRAMEBUFFER_COMPLETE, GL_INVALID_FRAMEBUFFER_OPERATION);
     }
 
@@ -1580,7 +1580,7 @@
 
     if (!has_client_vertex_arrays && !has_indirect_arrays) {
         // ALOGW("glDrawElements: no vertex arrays / buffers bound to the command\n");
-        GLenum status = ctx->m_glCheckFramebufferStatus_enc(self, GL_FRAMEBUFFER);
+        GLenum status = ctx->glCheckFramebufferStatus(self, GL_FRAMEBUFFER);
         SET_ERROR_IF(status != GL_FRAMEBUFFER_COMPLETE, GL_INVALID_FRAMEBUFFER_OPERATION);
     }
 
@@ -4581,7 +4581,7 @@
 
     if (!has_client_vertex_arrays && !has_indirect_arrays) {
         // ALOGW("glDrawElements: no vertex arrays / buffers bound to the command\n");
-        GLenum status = ctx->m_glCheckFramebufferStatus_enc(self, GL_FRAMEBUFFER);
+        GLenum status = ctx->glCheckFramebufferStatus(self, GL_FRAMEBUFFER);
         SET_ERROR_IF(status != GL_FRAMEBUFFER_COMPLETE, GL_INVALID_FRAMEBUFFER_OPERATION);
     }
 
@@ -4674,7 +4674,7 @@
 
     if (!has_client_vertex_arrays && !has_indirect_arrays) {
         // ALOGW("glDrawElements: no vertex arrays / buffers bound to the command\n");
-        GLenum status = ctx->m_glCheckFramebufferStatus_enc(self, GL_FRAMEBUFFER);
+        GLenum status = ctx->glCheckFramebufferStatus(self, GL_FRAMEBUFFER);
         SET_ERROR_IF(status != GL_FRAMEBUFFER_COMPLETE, GL_INVALID_FRAMEBUFFER_OPERATION);
     }