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

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

Issue 555020: Redesigned CommandBuffer and NPDevice3D interfaces (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 10 years, 11 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/gpu_processor.cc
===================================================================
--- gpu/command_buffer/service/gpu_processor.cc (revision 37414)
+++ gpu/command_buffer/service/gpu_processor.cc (working copy)
@@ -32,7 +32,8 @@
}
void GPUProcessor::ProcessCommands() {
- if (command_buffer_->GetErrorStatus())
+ CommandBuffer::State state = command_buffer_->GetState();
+ if (state.error != parse_error::kParseNoError)
return;
if (decoder_.get()) {
@@ -41,26 +42,15 @@
return;
}
- parser_->set_put(command_buffer_->GetPutOffset());
+ parser_->set_put(state.put_offset);
int commands_processed = 0;
while (commands_processed < commands_per_update_ && !parser_->IsEmpty()) {
parse_error::ParseError parse_error = parser_->ProcessCommand();
- switch (parse_error) {
- case parse_error::kParseUnknownCommand:
- case parse_error::kParseInvalidArguments:
- command_buffer_->SetParseError(parse_error);
- break;
-
- case parse_error::kParseInvalidSize:
- case parse_error::kParseOutOfBounds:
- command_buffer_->SetParseError(parse_error);
- command_buffer_->RaiseErrorStatus();
- return;
- case gpu::parse_error::kParseNoError:
- break;
+ if (parse_error != parse_error::kParseNoError) {
+ command_buffer_->SetParseError(parse_error);
+ return;
}
-
++commands_processed;
}
« no previous file with comments | « gpu/command_buffer/service/command_buffer_service_unittest.cc ('k') | gpu/command_buffer/service/gpu_processor_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698