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

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

Issue 888903002: Revert of Move AW renderer compositor context to gpu thread (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 11 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
« no previous file with comments | « no previous file | gpu/command_buffer/service/async_pixel_transfer_manager_android.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 75a82440a365ad3076e801b646a27ada20a17caf..4767b834a2d42ef15b211dcfa4b4896baf9fa961 100644
--- a/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
+++ b/content/browser/android/in_process/synchronous_compositor_factory_impl.cc
@@ -4,7 +4,6 @@
#include "content/browser/android/in_process/synchronous_compositor_factory_impl.h"
-#include "base/command_line.h"
#include "base/observer_list.h"
#include "content/browser/android/in_process/synchronous_compositor_external_begin_frame_source.h"
#include "content/browser/android/in_process/synchronous_compositor_impl.h"
@@ -14,7 +13,6 @@
#include "gpu/blink/webgraphicscontext3d_in_process_command_buffer_impl.h"
#include "gpu/command_buffer/client/gl_in_process_context.h"
#include "gpu/command_buffer/common/gles2_cmd_utils.h"
-#include "gpu/command_buffer/service/gpu_switches.h"
#include "ui/gl/android/surface_texture.h"
#include "ui/gl/gl_surface.h"
#include "ui/gl/gl_surface_stub.h"
@@ -66,8 +64,7 @@
scoped_ptr<gpu::GLInProcessContext> CreateContext(
scoped_refptr<gpu::InProcessCommandBuffer::Service> service,
- const gpu::GLInProcessContextSharedMemoryLimits& mem_limits,
- bool is_offscreen) {
+ const gpu::GLInProcessContextSharedMemoryLimits& mem_limits) {
const gfx::GpuPreference gpu_preference = gfx::PreferDiscreteGpu;
gpu::gles2::ContextCreationAttribHelper in_process_attribs;
WebGraphicsContext3DImpl::ConvertAttributes(
@@ -77,7 +74,7 @@
scoped_ptr<gpu::GLInProcessContext> context(gpu::GLInProcessContext::Create(
service,
NULL /* surface */,
- is_offscreen,
+ false /* is_offscreen */,
gfx::kNullAcceleratedWidget,
gfx::Size(1, 1),
NULL /* share_context */,
@@ -222,7 +219,7 @@
// pipeline is only one frame deep.
mem_limits.mapped_memory_reclaim_limit = 6 * 1024 * 1024;
return webkit::gpu::ContextProviderInProcess::Create(
- WrapContext(CreateContext(nullptr, mem_limits, true)),
+ WrapContext(CreateContext(service_, mem_limits)),
"Child-Compositor");
}
@@ -291,10 +288,9 @@
if (!video_context_provider_.get()) {
DCHECK(service_.get());
- // This needs to run in on-screen |service_| context due to SurfaceTexture
- // limitations.
- video_context_provider_ = new VideoContextProvider(CreateContext(
- service_, gpu::GLInProcessContextSharedMemoryLimits(), false));
+ video_context_provider_ = new VideoContextProvider(
+ CreateContext(service_,
+ gpu::GLInProcessContextSharedMemoryLimits()));
}
return video_context_provider_;
}
« no previous file with comments | « no previous file | gpu/command_buffer/service/async_pixel_transfer_manager_android.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698