Use gpu::SharedMemoryLimits for in process GL contexts.

android_webview/browser/aw_render_thread_context_provider.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "android_webview/browser/aw_render_thread_context_provider.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/lazy_instance.h"
 #include "base/trace_event/trace_event.h"
 #include "cc/output/managed_memory_policy.h"
 #include "gpu/blink/webgraphicscontext3d_impl.h"
 #include "gpu/command_buffer/client/gl_in_process_context.h"
 #include "gpu/command_buffer/client/gles2_implementation.h"
 #include "gpu/command_buffer/client/gles2_lib.h"
+#include "gpu/command_buffer/client/shared_memory_limits.h"
 #include "gpu/skia_bindings/gl_bindings_skia_cmd_buffer.h"
 #include "third_party/skia/include/gpu/GrContext.h"
 #include "third_party/skia/include/gpu/gl/GrGLInterface.h"
 
 namespace android_webview {
 
 // static
 scoped_refptr<AwRenderThreadContextProvider>
 AwRenderThreadContextProvider::Create(
     scoped_refptr<gfx::GLSurface> surface,
@@ skipping 17 matching lines @@
   // having them both in order to integrate its output with the content behind
   // it.
   attributes.alpha_size = 8;
   attributes.stencil_size = 8;
   // The depth buffer may exist due to having a stencil buffer, but we don't
   // need one, so use -1 for it.
   attributes.depth_size = -1;
   attributes.samples = 0;
   attributes.sample_buffers = 0;
   attributes.bind_generates_resource = false;
+
+  gpu::SharedMemoryLimits limits;
+  // TODO(boliu): Why?
+  limits.start_transfer_buffer_size = 4 * 1024 * 1024;

[danakj, 2016/04/20 02:13:53]

I took this from the old defaults which were different from the ones that were
in WebGraphicsContext3DCommandBufferImpl, for some reason?

[boliu, 2016/04/20 02:39:23]

This only runs the delegated compositor, which is draw quads only, and no
upload, right? Can we save some memory by lowering this to like something tiny?
That'd be great :p

[danakj, 2016/04/20 21:55:02]

Stealing from compositor_impl_android then.

+
   context_.reset(gpu::GLInProcessContext::Create(
       service, surface, surface->IsOffscreen(), gfx::kNullAcceleratedWidget,
       surface->GetSize(), nullptr /* share_context */, attributes,
-      gfx::PreferDiscreteGpu, gpu::GLInProcessContextSharedMemoryLimits(),
-      nullptr, nullptr));
+      gfx::PreferDiscreteGpu, limits, nullptr, nullptr));
 
   context_->GetImplementation()->SetLostContextCallback(base::Bind(
       &AwRenderThreadContextProvider::OnLostContext, base::Unretained(this)));
 }
 
 AwRenderThreadContextProvider::~AwRenderThreadContextProvider() {
   DCHECK(main_thread_checker_.CalledOnValidThread());
   if (gr_context_)
     gr_context_->releaseResourcesAndAbandonContext();
 }
@@ skipping 72 matching lines @@
 void AwRenderThreadContextProvider::OnLostContext() {
   DCHECK(main_thread_checker_.CalledOnValidThread());
 
   if (!lost_context_callback_.is_null())
     lost_context_callback_.Run();
   if (gr_context_)
     gr_context_->abandonContext();
 }
 
 }  // namespace android_webview