cc: Set the mapped memory reclaim limit for the renderer compositor on Android

content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc

Index: content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc
diff --git a/content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc b/content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc
index 80f4ebd043d36b6ccbb2052ebf9b4fa10564dac9..162eb1021867721abfcb6070f65decb7fb1a36ca 100644
--- a/content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc
+++ b/content/common/gpu/client/webgraphicscontext3d_command_buffer_impl.cc
@@ -481,11 +481,16 @@ bool WebGraphicsContext3DCommandBufferImpl::CreateContext(
     lock.reset();
   }
 
+  size_t gl_mapped_memory_limit = gpu::gles2::GLES2Implementation::kNoLimit;
+#if defined(OS_ANDROID)
+  gl_mapped_memory_limit = mapped_memory_limit_;
+#endif

[piman, 2013/08/30 03:31:07]

noooo!

This is taking the problem upside down.

You want to decide the amount of uploads the compositor is going to do per
frame, and from that, size the command buffer so that it's at least big enough
so that you don't have to stall.

So, instead of setting the limit on the context, then extracting that value in
the compositor and then ignoring the limit on !ANDROID, why don't we:
- decide the amount of uploads per frame. pass that explicitly to the compositor
- figure out the limit in the context. On Android we may want it tight because
we're memory constrained, so set the limit to the kMaxTransferBufferUsageBytes.
On other platforms, where we're not constrained, don't set the limit (or set it
to a larger value) so that we don't risk stalling.

In either case, this decision should be made in the embedder (RenderWidget, so
be it), not as an ifdef that affects all contexts for no good reason.

+
   if (!real_gl_->Initialize(
       start_transfer_buffer_size_,
       min_transfer_buffer_size_,
       max_transfer_buffer_size_,
-      mapped_memory_limit_)) {
+      gl_mapped_memory_limit)) {
     return false;
   }