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

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

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: Created 4 years, 2 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 0ce169540b3cae2c0e67d865f585d0d48d8763a3..b98d51dc96afc63fa88e157fe57ceb950df6ddbf 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1634,10 +1634,9 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
view_->DidReceiveRendererFrame();
} else {
cc::ReturnedResourceArray resources;
- if (frame.delegated_frame_data) {
- cc::TransferableResource::ReturnResources(
- frame.delegated_frame_data->resource_list, &resources);
- }
+ if (!frame.IsEmpty())
piman 2016/10/25 22:57:31 nit: you can skip this test, ReturnResources is a
Saman Sami 2016/10/28 16:47:35 Done.
+ cc::TransferableResource::ReturnResources(frame.resource_list,
+ &resources);
SendReclaimCompositorResources(routing_id_, compositor_frame_sink_id,
process_->GetID(), true /* is_swap_ack */,
resources);

Powered by Google App Engine
This is Rietveld 408576698