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

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

Issue 116863003: gpu: Reuse transfer buffers more aggresively (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Async upload token part of existing Async command; use separate shared memory to sync async upload … Created 6 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/async_pixel_transfer_manager_share_group.cc
diff --git a/gpu/command_buffer/service/async_pixel_transfer_manager_share_group.cc b/gpu/command_buffer/service/async_pixel_transfer_manager_share_group.cc
index dcdab1c69adb5dd2a4c9be58a9351ece0b258849..1ef75ee5af8428b1e8b6c2f010d86da273bd6369 100644
--- a/gpu/command_buffer/service/async_pixel_transfer_manager_share_group.cc
+++ b/gpu/command_buffer/service/async_pixel_transfer_manager_share_group.cc
@@ -536,6 +536,11 @@ void AsyncPixelTransferManagerShareGroup::BindCompletedAsyncTransfers() {
}
}
+void AsyncPixelTransferManagerShareGroup::AsyncRun(
+ const base::Closure& callback) {
+ transfer_message_loop_proxy()->PostTask(FROM_HERE, callback);
+}
+
void AsyncPixelTransferManagerShareGroup::AsyncNotifyCompletion(
const AsyncMemoryParams& mem_params,
AsyncPixelTransferCompletionObserver* observer) {
@@ -544,8 +549,7 @@ void AsyncPixelTransferManagerShareGroup::AsyncNotifyCompletion(
mem_params.shm_size);
// Post a PerformNotifyCompletion task to the upload thread. This task
// will run after all async transfers are complete.
- transfer_message_loop_proxy()->PostTask(
- FROM_HERE,
+ AsyncRun(
base::Bind(&PerformNotifyCompletion,
mem_params,
base::Owned(

Powered by Google App Engine
This is Rietveld 408576698