With latch support and higher level frame throttling, we no longer need to limit SwapBuffers calls by using glSet/TestFenceNV in GpuScheduler.
BUG=79940
TEST=open page with CSS 3D transform animation; open about:gpu and start tracing; verify that between frames, there are not many short calls to ProcessCommands
Review URL: http://codereview.chromium.org/6873095
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@82197 0039d316-1c4b-4281-b951-d872f2087c98
diff --git a/gpu/command_buffer/service/gles2_cmd_decoder.cc b/gpu/command_buffer/service/gles2_cmd_decoder.cc
index db58132..42f9a2b 100644
--- a/gpu/command_buffer/service/gles2_cmd_decoder.cc
+++ b/gpu/command_buffer/service/gles2_cmd_decoder.cc
@@ -6341,7 +6341,7 @@
if (swap_buffers_callback_.get()) {
swap_buffers_callback_->Run();
}
- return error::kThrottle;
+ return error::kNoError;
} else {
ScopedFrameBufferBinder binder(this,
offscreen_target_frame_buffer_->id());
@@ -6363,7 +6363,7 @@
if (swap_buffers_callback_.get()) {
swap_buffers_callback_->Run();
}
- return error::kThrottle;
+ return error::kNoError;
}
} else {
if (!context_->SwapBuffers()) {
@@ -6376,14 +6376,7 @@
swap_buffers_callback_->Run();
}
- // Do not throttle SwapBuffers by returning kThrottle. The intent of
- // throttling the offscreen command buffers to a fixed number of frames
- // ahead is to prevent them from rendering faster than they can be
- // presented, not to limit the rate at which we present.
- //
- // This does not hold for ANGLE, possibly because all the GL contexts in a
- // share group are actually one D3D device. Found by trial and error.
- return IsAngle() ? error::kThrottle : error::kNoError;
+ return error::kNoError;
}
error::Error GLES2DecoderImpl::HandleSetLatchCHROMIUM(
