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

Unified Diff: services/gfx/compositor/backend/gpu_output.cc

Issue 1682113003: Mojo C++ bindings: Generate InterfaceHandle<> instead of InterfacePtr<>. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: rebase ontop of master, address trung's comments Created 4 years, 10 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 | « services/gfx/compositor/backend/gpu_output.h ('k') | services/gfx/compositor/backend/gpu_rasterizer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: services/gfx/compositor/backend/gpu_output.cc
diff --git a/services/gfx/compositor/backend/gpu_output.cc b/services/gfx/compositor/backend/gpu_output.cc
index e5fe85e790b6068c695fd166f56461ac97748d2d..950de50b0151ba9d1f9c476f6c96da64e9131f9e 100644
--- a/services/gfx/compositor/backend/gpu_output.cc
+++ b/services/gfx/compositor/backend/gpu_output.cc
@@ -22,13 +22,14 @@ static scoped_ptr<base::MessagePump> CreateMessagePumpMojo() {
base::MessageLoop::TYPE_DEFAULT);
}
-GpuOutput::GpuOutput(mojo::ContextProviderPtr context_provider,
- const SchedulerCallbacks& scheduler_callbacks,
- const base::Closure& error_callback)
+GpuOutput::GpuOutput(
+ mojo::InterfaceHandle<mojo::ContextProvider> context_provider,
+ const SchedulerCallbacks& scheduler_callbacks,
+ const base::Closure& error_callback)
: frame_queue_(std::make_shared<FrameQueue>()),
- scheduler_(std::make_shared<VsyncScheduler>(base::MessageLoop::current()
- ->task_runner(),
- scheduler_callbacks)),
+ scheduler_(std::make_shared<VsyncScheduler>(
+ base::MessageLoop::current()->task_runner(),
+ scheduler_callbacks)),
rasterizer_delegate_(
make_scoped_ptr(new RasterizerDelegate(frame_queue_))) {
DCHECK(context_provider);
@@ -44,9 +45,8 @@ GpuOutput::GpuOutput(mojo::ContextProviderPtr context_provider,
FROM_HERE,
base::Bind(&RasterizerDelegate::CreateRasterizer,
base::Unretained(rasterizer_delegate_.get()),
- base::Passed(context_provider.PassInterfaceHandle()),
- scheduler_, base::MessageLoop::current()->task_runner(),
- error_callback));
+ base::Passed(std::move(context_provider)), scheduler_,
+ base::MessageLoop::current()->task_runner(), error_callback));
}
GpuOutput::~GpuOutput() {
« no previous file with comments | « services/gfx/compositor/backend/gpu_output.h ('k') | services/gfx/compositor/backend/gpu_rasterizer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698