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

Unified Diff: content/renderer/android/synchronous_compositor_frame_sink.cc

Issue 2503203002: Revert "Getting rid of DelegatedFrameData" (Closed)
Patch Set: Created 4 years, 1 month 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 | « content/browser/renderer_host/render_widget_host_view_mac.mm ('k') | services/ui/demo/bitmap_uploader.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/android/synchronous_compositor_frame_sink.cc
diff --git a/content/renderer/android/synchronous_compositor_frame_sink.cc b/content/renderer/android/synchronous_compositor_frame_sink.cc
index 4113e6210d05e592ebc7534a07c63e7f8867d2cf..48be138c154e7585e4388d311b0102cb0c7f555b 100644
--- a/content/renderer/android/synchronous_compositor_frame_sink.cc
+++ b/content/renderer/android/synchronous_compositor_frame_sink.cc
@@ -221,7 +221,7 @@ void SynchronousCompositorFrameSink::SubmitCompositorFrame(
DCHECK(sync_client_);
if (fallback_tick_running_) {
- DCHECK(frame.resource_list.empty());
+ DCHECK(frame.delegated_frame_data->resource_list.empty());
cc::ReturnedResourceArray return_resources;
ReturnResources(return_resources);
did_submit_frame_ = true;
@@ -248,7 +248,8 @@ void SynchronousCompositorFrameSink::SubmitCompositorFrame(
// The layer compositor should be giving a frame that covers the
// |sw_viewport_for_current_draw_| but at 0,0.
gfx::Size child_size = sw_viewport_for_current_draw_.size();
- DCHECK(gfx::Rect(child_size) == frame.render_pass_list.back()->output_rect);
+ DCHECK(gfx::Rect(child_size) ==
+ frame.delegated_frame_data->render_pass_list.back()->output_rect);
// Make a size that covers from 0,0 and includes the area coming from the
// layer compositor.
@@ -268,10 +269,14 @@ void SynchronousCompositorFrameSink::SubmitCompositorFrame(
// the CompositorFrameSink client too? (We'd have to do the same for
// hardware frames in SurfacesInstance?)
cc::CompositorFrame embed_frame;
- embed_frame.render_pass_list.push_back(cc::RenderPass::Create());
+ embed_frame.delegated_frame_data =
+ base::MakeUnique<cc::DelegatedFrameData>();
+ embed_frame.delegated_frame_data->render_pass_list.push_back(
+ cc::RenderPass::Create());
// The embedding RenderPass covers the entire Display's area.
- const auto& embed_render_pass = embed_frame.render_pass_list.back();
+ const auto& embed_render_pass =
+ embed_frame.delegated_frame_data->render_pass_list.back();
embed_render_pass->SetAll(cc::RenderPassId(1, 1), gfx::Rect(display_size),
gfx::Rect(display_size), gfx::Transform(), false);
« no previous file with comments | « content/browser/renderer_host/render_widget_host_view_mac.mm ('k') | services/ui/demo/bitmap_uploader.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698