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

Unified Diff: content/browser/frame_host/render_widget_host_view_child_frame.cc

Issue 2774373002: Use MojoCompositorFrameSink in RendererCompositorFrameSink (Closed)
Patch Set: Fixed mac Created 3 years, 8 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/frame_host/render_widget_host_view_child_frame.cc
diff --git a/content/browser/frame_host/render_widget_host_view_child_frame.cc b/content/browser/frame_host/render_widget_host_view_child_frame.cc
index 4a33acda21c830f7bd97ef3cb61b7547ca28391d..98fe44959abb57fe810dc1e3af1833630beb091e 100644
--- a/content/browser/frame_host/render_widget_host_view_child_frame.cc
+++ b/content/browser/frame_host/render_widget_host_view_child_frame.cc
@@ -347,14 +347,14 @@ void RenderWidgetHostViewChildFrame::GestureEventAck(
}
void RenderWidgetHostViewChildFrame::DidReceiveCompositorFrameAck() {
- if (!host_)
- return;
- host_->SendReclaimCompositorResources(true /* is_swap_ack */,
- cc::ReturnedResourceArray());
+ DCHECK(renderer_compositor_frame_sink_);
+ renderer_compositor_frame_sink_->DidReceiveCompositorFrameAck();
}
-void RenderWidgetHostViewChildFrame::DidCreateNewRendererCompositorFrameSink() {
+void RenderWidgetHostViewChildFrame::DidCreateNewRendererCompositorFrameSink(
+ cc::mojom::MojoCompositorFrameSinkClient* renderer_compositor_frame_sink) {
ResetCompositorFrameSinkSupport();
+ renderer_compositor_frame_sink_ = renderer_compositor_frame_sink;
CreateCompositorFrameSinkSupport();
has_frame_ = false;
}
@@ -617,14 +617,12 @@ bool RenderWidgetHostViewChildFrame::HasAcceleratedSurface(
void RenderWidgetHostViewChildFrame::ReclaimResources(
const cc::ReturnedResourceArray& resources) {
- if (!host_)
- return;
- host_->SendReclaimCompositorResources(false /* is_swap_ack */, resources);
+ renderer_compositor_frame_sink_->ReclaimResources(resources);
}
void RenderWidgetHostViewChildFrame::OnBeginFrame(
const cc::BeginFrameArgs& args) {
- host_->Send(new ViewMsg_BeginFrame(host_->GetRoutingID(), args));
+ renderer_compositor_frame_sink_->OnBeginFrame(args);
}
void RenderWidgetHostViewChildFrame::SetNeedsBeginFrames(

Powered by Google App Engine
This is Rietveld 408576698