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

Unified Diff: services/ui/ws/platform_display_default.cc

Issue 2890913002: Add CompositorFrameSinkClientBinding To Be Used By FrameGenerator (Closed)
Patch Set: virtual destructor Created 3 years, 7 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: services/ui/ws/platform_display_default.cc
diff --git a/services/ui/ws/platform_display_default.cc b/services/ui/ws/platform_display_default.cc
index b879e33517207034f050e1fbb4829724526a6401..42e7d1302f62f47ee3cb58157b6e0b1d17b95671 100644
--- a/services/ui/ws/platform_display_default.cc
+++ b/services/ui/ws/platform_display_default.cc
@@ -262,13 +262,13 @@ void PlatformDisplayDefault::OnAcceleratedWidgetAvailable(
std::move(compositor_frame_sink_client),
mojo::MakeRequest(&display_private));
- auto display_client_compositor_frame_sink =
- base::MakeUnique<DisplayClientCompositorFrameSink>(
Fady Samuel 2017/05/17 20:08:18 Do you still need DisplayClientCompositorFrameSink
Alex Z. 2017/05/17 20:21:18 Done.
- root_window_->frame_sink_id(), std::move(compositor_frame_sink),
- std::move(display_private),
- std::move(compositor_frame_sink_client_request));
- frame_generator_ = base::MakeUnique<FrameGenerator>(
- std::move(display_client_compositor_frame_sink));
+ frame_generator_ = base::MakeUnique<FrameGenerator>();
+ auto frame_sink_client_binding =
+ base::MakeUnique<DefaultCompositorFrameSinkClientBinding>(
+ frame_generator_.get(),
+ std::move(compositor_frame_sink_client_request),
+ std::move(compositor_frame_sink), std::move(display_private));
+ frame_generator_->Bind(std::move(frame_sink_client_binding));
frame_generator_->OnWindowSizeChanged(root_window_->bounds().size());
frame_generator_->SetDeviceScaleFactor(metrics_.device_scale_factor);
}
« services/ui/ws/frame_generator_unittest.cc ('K') | « services/ui/ws/frame_generator_unittest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698