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

Unified Diff: content/browser/android/in_process/synchronous_compositor_factory_impl.cc

Issue 540143002: InProcessGL: Share MailboxManager per service (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@clankium
Patch Set: Created 6 years, 3 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: content/browser/android/in_process/synchronous_compositor_factory_impl.cc
diff --git a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
index b7a2cc2ed9aab0f77526c2045d4f0d626283015a..b736edb9078b028c6c016c74ff0173a2e969f56f 100644
--- a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
+++ b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
@@ -60,7 +60,6 @@ scoped_ptr<gpu::GLInProcessContext> CreateOffscreenContext(
scoped_ptr<gpu::GLInProcessContext> CreateContext(
scoped_refptr<gpu::InProcessCommandBuffer::Service> service,
- gpu::GLInProcessContext* share_context,
const gpu::GLInProcessContextSharedMemoryLimits& mem_limits) {
const gfx::GpuPreference gpu_preference = gfx::PreferDiscreteGpu;
gpu::gles2::ContextCreationAttribHelper in_process_attribs;
@@ -74,7 +73,7 @@ scoped_ptr<gpu::GLInProcessContext> CreateContext(
false /* is_offscreen */,
gfx::kNullAcceleratedWidget,
gfx::Size(1, 1),
- share_context,
+ NULL /* share_context */,
false /* share_resources */,
in_process_attribs,
gpu_preference,
@@ -187,24 +186,15 @@ scoped_refptr<cc::ContextProvider> SynchronousCompositorFactoryImpl::
CreateOnscreenContextProviderForCompositorThread() {
DCHECK(service_);
- if (!share_context_.get()) {
- share_context_ = CreateContext(
- service_, NULL, gpu::GLInProcessContextSharedMemoryLimits());
- }
gpu::GLInProcessContextSharedMemoryLimits mem_limits;
// This is half of what RenderWidget uses because synchronous compositor
// pipeline is only one frame deep.
mem_limits.mapped_memory_reclaim_limit = 6 * 1024 * 1024;
return webkit::gpu::ContextProviderInProcess::Create(
- WrapContext(CreateContext(service_, share_context_.get(), mem_limits)),
+ WrapContext(CreateContext(service_, mem_limits)),
"Child-Compositor");
}
-gpu::GLInProcessContext* SynchronousCompositorFactoryImpl::GetShareContext() {
- DCHECK(share_context_.get());
- return share_context_.get();
-}
-
scoped_refptr<StreamTextureFactory>
SynchronousCompositorFactoryImpl::CreateStreamTextureFactory(int frame_id) {
scoped_refptr<StreamTextureFactorySynchronousImpl> factory(
@@ -252,11 +242,9 @@ SynchronousCompositorFactoryImpl::TryCreateStreamTextureFactory() {
if (!video_context_provider_) {
DCHECK(service_);
- DCHECK(share_context_.get());
video_context_provider_ = new VideoContextProvider(
CreateContext(service_,
- share_context_.get(),
gpu::GLInProcessContextSharedMemoryLimits()));
}
return video_context_provider_;

Powered by Google App Engine
This is Rietveld 408576698