Make cc::CompositorFrames movable [Part 1 of 2]

cc/output/delegating_renderer.cc

Index: cc/output/delegating_renderer.cc
diff --git a/cc/output/delegating_renderer.cc b/cc/output/delegating_renderer.cc
index c063c7f608dfbfdbda048ebc7f942af291395b9b..9093a0f0213f554598b5fc1c434f9914a87edc74 100644
--- a/cc/output/delegating_renderer.cc
+++ b/cc/output/delegating_renderer.cc
@@ -102,9 +102,9 @@ void DelegatingRenderer::DrawFrame(RenderPassList* render_passes_in_draw_order,
 void DelegatingRenderer::SwapBuffers(const CompositorFrameMetadata& metadata) {
   TRACE_EVENT0("cc,benchmark", "DelegatingRenderer::SwapBuffers");
   CompositorFrame compositor_frame;
-  compositor_frame.metadata = metadata;
+  compositor_frame.metadata = CompositorFrameMetadata(metadata);

[danakj, 2016/06/24 18:35:10]

what is this? do you need to define a copy operator= to match the constructor?

or make it move-only and pass it by value to here?

[Fady Samuel, 2016/06/24 20:00:24]

Done.

   compositor_frame.delegated_frame_data = std::move(delegated_frame_data_);
-  output_surface_->SwapBuffers(&compositor_frame);
+  output_surface_->SwapBuffers(std::move(compositor_frame));
 }
 
 void DelegatingRenderer::ReceiveSwapBuffersAck(