Commit 83e5bb97 by Lubosz Sarnecki Committed by Commit Bot

FrameCapture: Don't capture shader functions on GLES1.

Since an internal default shader is created and captured here in the GLES1 case, we need to check for the context version to skip capturing glUseProgram and emit an update call in GLES1 traces. Bug: angleproject:5893 Change-Id: Iac9bc05234fbc87c1f64264b5c12b9b117522bad Reviewed-on: https://chromium-review.googlesource.com/c/angle/angle/+/2877236 Commit-Queue: Lubosz Sarnecki <lubosz.sarnecki@collabora.com> Reviewed-by: 's avatarJamie Madill <jmadill@chromium.org> Reviewed-by: 's avatarCody Northrop <cnorthrop@google.com>
parent 125f128e
...@@ -3004,7 +3004,7 @@ void CaptureMidExecutionSetup(const gl::Context *context, ...@@ -3004,7 +3004,7 @@ void CaptureMidExecutionSetup(const gl::Context *context,
// For now we assume the installed program executable is the same as the current program. // For now we assume the installed program executable is the same as the current program.
// TODO(jmadill): Handle installed program executable. http://anglebug.com/3662 // TODO(jmadill): Handle installed program executable. http://anglebug.com/3662
if (apiState.getProgram()) if (apiState.getProgram() && !context->isGLES1())
{ {
cap(CaptureUseProgram(replayState, true, apiState.getProgram()->id())); cap(CaptureUseProgram(replayState, true, apiState.getProgram()->id()));
CaptureUpdateCurrentProgram(setupCalls->back(), setupCalls); CaptureUpdateCurrentProgram(setupCalls->back(), setupCalls);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment