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

Unified Diff: blimp/client/support/compositor/blimp_context_provider.cc

Issue 2286873003: Provide TaskRunner to ContextCacheController (Closed)
Patch Set: Created 4 years, 4 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: blimp/client/support/compositor/blimp_context_provider.cc
diff --git a/blimp/client/support/compositor/blimp_context_provider.cc b/blimp/client/support/compositor/blimp_context_provider.cc
index 83874cbc18c56255429ecfc2a072bfc2dc5ebfff..a2f9975c472de6512e1cd368a83dbac33006297b 100644
--- a/blimp/client/support/compositor/blimp_context_provider.cc
+++ b/blimp/client/support/compositor/blimp_context_provider.cc
@@ -7,6 +7,7 @@
#include "base/bind.h"
#include "base/callback_helpers.h"
#include "base/lazy_instance.h"
+#include "base/threading/thread_task_runner_handle.h"
#include "cc/output/context_cache_controller.h"
#include "gpu/command_buffer/client/gl_in_process_context.h"
#include "gpu/command_buffer/client/gles2_implementation.h"
@@ -42,16 +43,18 @@ BlimpContextProvider::BlimpContextProvider(
attribs_for_gles2.context_type = gpu::gles2::CONTEXT_TYPE_OPENGLES2;
attribs_for_gles2.lose_context_when_out_of_memory = true;
+ auto task_runner = base::ThreadTaskRunnerHandle::Get();
+
context_.reset(gpu::GLInProcessContext::Create(
nullptr /* service */, nullptr /* surface */,
widget == gfx::kNullAcceleratedWidget /* is_offscreen */, widget,
nullptr /* share_context */, attribs_for_gles2, gpu::SharedMemoryLimits(),
- gpu_memory_buffer_manager, nullptr /* memory_limits */));
+ gpu_memory_buffer_manager, nullptr /* memory_limits */, task_runner));
context_->GetImplementation()->SetLostContextCallback(
base::Bind(&BlimpContextProvider::OnLostContext, base::Unretained(this)));
- cache_controller_.reset(
- new cc::ContextCacheController(context_->GetImplementation()));
+ cache_controller_.reset(new cc::ContextCacheController(
+ context_->GetImplementation(), std::move(task_runner)));
}
BlimpContextProvider::~BlimpContextProvider() {

Powered by Google App Engine
This is Rietveld 408576698