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

Unified Diff: webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc

Issue 19522006: GLInProcessContext: support async flushes and dedicated GPU thread (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 5 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: webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc
diff --git a/webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc b/webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc
index 7f5f17018e175b0957de553413a1b485c105c6bb..57767ec07397880cdeb510260e92cfad9e699d22 100644
--- a/webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc
+++ b/webkit/common/gpu/webgraphicscontext3d_in_process_command_buffer_impl.cc
@@ -1163,11 +1163,6 @@ void WebGraphicsContext3DInProcessCommandBufferImpl::signalSyncPoint(
// Take ownership of the callback.
context_->SignalSyncPoint(
sync_point, base::Bind(&OnSignalSyncPoint, base::Owned(callback)));
- // Make sure we have something to flush, or shallowFlushCHROMIUM()
- // doesn't do anything.
- gl_->helper()->Noop(1);
- // This fill force PumpCommands() to run which is what triggers the signal.
- shallowFlushCHROMIUM();
}
void WebGraphicsContext3DInProcessCommandBufferImpl::signalQuery(
@@ -1182,7 +1177,6 @@ void WebGraphicsContext3DInProcessCommandBufferImpl::loseContextCHROMIUM(
WGC3Denum current, WGC3Denum other) {
gl_->LoseContextCHROMIUM(current, other);
gl_->ShallowFlushCHROMIUM();
- DCHECK(isContextLost());
}
DELEGATE_TO_GL_9(asyncTexImage2DCHROMIUM, AsyncTexImage2DCHROMIUM,
« gpu/command_buffer/service/in_process_command_buffer.cc ('K') | « gpu/command_buffer_service.gypi ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698