Re-land: Mark async texture uploads as completed from the upload thread.

ui/gl/async_pixel_transfer_delegate_stub.cc

Index: ui/gl/async_pixel_transfer_delegate_stub.cc
diff --git a/ui/gl/async_pixel_transfer_delegate_stub.cc b/ui/gl/async_pixel_transfer_delegate_stub.cc
index 86f068735a49c237fb88f23a3cdc8cdffd3a7fe4..1721188229da0f6f946f54282d98a6c21cb4f6ac 100644
--- a/ui/gl/async_pixel_transfer_delegate_stub.cc
+++ b/ui/gl/async_pixel_transfer_delegate_stub.cc
@@ -6,6 +6,7 @@
 
 #include "base/shared_memory.h"
 #include "build/build_config.h"
+#include "gpu/command_buffer/common/gles2_cmd_format.h"
 #include "ui/gl/gl_bindings.h"
 
 using base::SharedMemory;
@@ -77,8 +78,16 @@ AsyncPixelTransferState*
 }
 
 void AsyncPixelTransferDelegateStub::AsyncNotifyCompletion(
-    const base::Closure& task) {
-  task.Run();
+    const AsyncMemoryParams& mem_params,
+    uint32 submit_count) {
+  gpu::gles2::QuerySync* sync = static_cast<gpu::gles2::QuerySync*>(

[epenner, 2013/02/07 20:22:56]

See my earlier comment. I think the layering violation can be avoided with a
callback.

+      GetAddress(mem_params.shared_memory,
+                 mem_params.shm_size,
+                 mem_params.shm_data_offset,
+                 mem_params.shm_data_size));
+  if (!sync)
+    return;
+  sync->process_count = submit_count;
 }
 
 void AsyncPixelTransferDelegateStub::AsyncTexImage2D(