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

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

Issue 951673002: Revert "Pull chromium at 2c3ffb2355a27c32f45e508ef861416b820c823b" (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 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/shader_manager.cc
diff --git a/gpu/command_buffer/service/shader_manager.cc b/gpu/command_buffer/service/shader_manager.cc
index d88a628874ab44fd25e9cc844e60d3ce48a8e884..055cd707a539b3db87f6a1da81bb3476293ce5fb 100644
--- a/gpu/command_buffer/service/shader_manager.cc
+++ b/gpu/command_buffer/service/shader_manager.cc
@@ -81,18 +81,15 @@ void Shader::DoCompile() {
glGetShaderiv(service_id_,
GL_TRANSLATED_SHADER_SOURCE_LENGTH_ANGLE,
&max_len);
- source_for_driver = "\0";
translated_source_.resize(max_len);
- if (max_len) {
- GLint len = 0;
- glGetTranslatedShaderSourceANGLE(
- service_id_, translated_source_.size(),
- &len, &translated_source_.at(0));
- DCHECK(max_len == 0 || len < max_len);
- DCHECK(len == 0 || translated_source_[len] == '\0');
- translated_source_.resize(len);
- source_for_driver = translated_source_.c_str();
- }
+ GLint len = 0;
+ glGetTranslatedShaderSourceANGLE(
+ service_id_, translated_source_.size(),
+ &len, &translated_source_.at(0));
+ DCHECK(max_len == 0 || len < max_len);
+ DCHECK(len == 0 || translated_source_[len] == '\0');
+ translated_source_.resize(len);
+ source_for_driver = translated_source_.c_str();
}
GLint status = GL_FALSE;
@@ -100,21 +97,18 @@ void Shader::DoCompile() {
if (status == GL_TRUE) {
valid_ = true;
} else {
- valid_ = false;
-
// We cannot reach here if we are using the shader translator.
// All invalid shaders must be rejected by the translator.
// All translated shaders must compile.
GLint max_len = 0;
glGetShaderiv(service_id_, GL_INFO_LOG_LENGTH, &max_len);
log_info_.resize(max_len);
- if (max_len) {
- GLint len = 0;
- glGetShaderInfoLog(service_id_, log_info_.size(), &len, &log_info_.at(0));
- DCHECK(max_len == 0 || len < max_len);
- DCHECK(len == 0 || log_info_[len] == '\0');
- log_info_.resize(len);
- }
+ GLint len = 0;
+ glGetShaderInfoLog(service_id_, log_info_.size(), &len, &log_info_.at(0));
+ DCHECK(max_len == 0 || len < max_len);
+ DCHECK(len == 0 || log_info_[len] == '\0');
+ valid_ = false;
+ log_info_.resize(len);
LOG_IF(ERROR, translator)
<< "Shader translator allowed/produced an invalid shader "
<< "unless the driver is buggy:"
« no previous file with comments | « gpu/command_buffer/service/mailbox_manager_unittest.cc ('k') | gpu/command_buffer/service/texture_definition.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698