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

Unified Diff: gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc

Issue 2831733003: Fix blits from multisampled renderbuffers to alpha:false WebGL back buffer. (Closed)
Patch Set: Add PLATFORM_EXPORT to fix link failure on Windows. Created 3 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: gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc
diff --git a/gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc b/gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc
index 9bfcb6b2750c7578ba5e0b76107cdb2812d64803..3600f809af1e651a59682c19bd055293b4d035ff 100644
--- a/gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc
+++ b/gpu/command_buffer/service/gles2_cmd_decoder_passthrough_doers.cc
@@ -3485,22 +3485,15 @@ error::Error GLES2DecoderPassthroughImpl::DoBindUniformLocationCHROMIUM(
error::Error GLES2DecoderPassthroughImpl::DoBindTexImage2DCHROMIUM(
GLenum target,
GLint imageId) {
- if (target != GL_TEXTURE_2D) {
- InsertError(GL_INVALID_ENUM, "Invalid target");
- return error::kNoError;
- }
-
- gl::GLImage* image = image_manager_->LookupImage(imageId);
- if (image == nullptr) {
- InsertError(GL_INVALID_OPERATION, "No image found with the given ID");
- return error::kNoError;
- }
-
- if (!image->BindTexImage(target)) {
- image->CopyTexImage(target);
- }
+ return BindTexImage2DCHROMIUMImpl(target, 0, imageId);
+}
- return error::kNoError;
+error::Error
+GLES2DecoderPassthroughImpl::DoBindTexImage2DWithInternalformatCHROMIUM(
+ GLenum target,
+ GLenum internalformat,
+ GLint imageId) {
+ return BindTexImage2DCHROMIUMImpl(target, internalformat, imageId);
}
error::Error GLES2DecoderPassthroughImpl::DoReleaseTexImage2DCHROMIUM(

Powered by Google App Engine
This is Rietveld 408576698