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

Unified Diff: content/browser/renderer_host/render_widget_host_impl.cc

Issue 2728183002: RendererCompositorFrameSink should handle local surface id allocation (Closed)
Patch Set: Added comment Created 3 years, 9 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/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index e0540017123d77acb24a7e675abe5d4137b8b1be..8af09c3e5022532e2b4be2d7504e793322e20134 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1810,10 +1810,11 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
ViewHostMsg_SwapCompositorFrame::Param param;
if (!ViewHostMsg_SwapCompositorFrame::Read(&message, &param))
return false;
- cc::CompositorFrame frame(std::move(std::get<1>(param)));
uint32_t compositor_frame_sink_id = std::get<0>(param);
+ cc::LocalSurfaceId local_surface_id = std::get<1>(param);
+ cc::CompositorFrame frame(std::move(std::get<2>(param)));
std::vector<IPC::Message> messages_to_deliver_with_frame;
- messages_to_deliver_with_frame.swap(std::get<2>(param));
+ messages_to_deliver_with_frame.swap(std::get<3>(param));
if (!ui::LatencyInfo::Verify(frame.metadata.latency_info,
"RenderWidgetHostImpl::OnSwapCompositorFrame")) {
@@ -1834,7 +1835,8 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
// compositor frame can arrive before the navigation commit message that
// updates that value.
if (view_ && frame.metadata.content_source_id >= current_content_source_id_) {
- view_->OnSwapCompositorFrame(compositor_frame_sink_id, std::move(frame));
+ view_->OnSwapCompositorFrame(compositor_frame_sink_id, local_surface_id,
+ std::move(frame));
view_->DidReceiveRendererFrame();
} else {
cc::ReturnedResourceArray resources;

Powered by Google App Engine
This is Rietveld 408576698