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

cc/output/delegating_renderer_unittest.cc

Index: cc/output/delegating_renderer_unittest.cc
diff --git a/cc/output/delegating_renderer_unittest.cc b/cc/output/delegating_renderer_unittest.cc
index 86e9e31f32b37bf8c95d14b3f8017437c737eec2..39d1af9260c4ce0fef009c35318b467524733855 100644
--- a/cc/output/delegating_renderer_unittest.cc
+++ b/cc/output/delegating_renderer_unittest.cc
@@ -44,11 +44,11 @@ class DelegatingRendererTestDraw : public DelegatingRendererTest {
                                    DrawResult draw_result) override {
     EXPECT_EQ(0u, output_surface_->num_sent_frames());
 
-    const CompositorFrame& last_frame = output_surface_->last_sent_frame();
-    EXPECT_FALSE(last_frame.delegated_frame_data);
-    EXPECT_FALSE(last_frame.gl_frame_data);
-    EXPECT_EQ(0.f, last_frame.metadata.min_page_scale_factor);
-    EXPECT_EQ(0.f, last_frame.metadata.max_page_scale_factor);
+    const CompositorFrame* last_frame = output_surface_->last_sent_frame();
+    EXPECT_FALSE(last_frame->delegated_frame_data);
+    EXPECT_FALSE(last_frame->gl_frame_data);
+    EXPECT_EQ(0.f, last_frame->metadata.min_page_scale_factor);
+    EXPECT_EQ(0.f, last_frame->metadata.max_page_scale_factor);
     return DRAW_SUCCESS;
   }
 
@@ -60,18 +60,18 @@ class DelegatingRendererTestDraw : public DelegatingRendererTest {
     EXPECT_TRUE(result);
     EXPECT_EQ(1u, output_surface_->num_sent_frames());
 
-    const CompositorFrame& last_frame = output_surface_->last_sent_frame();
-    DelegatedFrameData* last_frame_data = last_frame.delegated_frame_data.get();
-    ASSERT_TRUE(last_frame.delegated_frame_data);
-    EXPECT_FALSE(last_frame.gl_frame_data);
+    const CompositorFrame* last_frame = output_surface_->last_sent_frame();
+    DelegatedFrameData* last_frame_data =
+        last_frame->delegated_frame_data.get();
+    ASSERT_TRUE(last_frame->delegated_frame_data);
+    EXPECT_FALSE(last_frame->gl_frame_data);
     EXPECT_EQ(host_impl->DeviceViewport().ToString(),
               last_frame_data->render_pass_list.back()->output_rect.ToString());
-    EXPECT_EQ(0.5f, last_frame.metadata.min_page_scale_factor);
-    EXPECT_EQ(4.f, last_frame.metadata.max_page_scale_factor);
+    EXPECT_EQ(0.5f, last_frame->metadata.min_page_scale_factor);
+    EXPECT_EQ(4.f, last_frame->metadata.max_page_scale_factor);
 
-    EXPECT_EQ(
-        0u, last_frame.delegated_frame_data->resource_list.size());
-    EXPECT_EQ(1u, last_frame.delegated_frame_data->render_pass_list.size());
+    EXPECT_EQ(0u, last_frame->delegated_frame_data->resource_list.size());
+    EXPECT_EQ(1u, last_frame->delegated_frame_data->render_pass_list.size());
 
     EndTest();
   }
@@ -112,16 +112,16 @@ class DelegatingRendererTestResources : public DelegatingRendererTest {
     EXPECT_TRUE(result);
     EXPECT_EQ(1u, output_surface_->num_sent_frames());
 
-    const CompositorFrame& last_frame = output_surface_->last_sent_frame();
-    ASSERT_TRUE(last_frame.delegated_frame_data);
+    CompositorFrame* last_frame = output_surface_->last_sent_frame();

[danakj, 2016/06/24 20:20:43]

nit: can stay const

[Fady Samuel, 2016/06/24 21:31:15]

Done.

+    ASSERT_TRUE(last_frame->delegated_frame_data);
 
-    EXPECT_EQ(2u, last_frame.delegated_frame_data->render_pass_list.size());
+    EXPECT_EQ(2u, last_frame->delegated_frame_data->render_pass_list.size());
     // Each render pass has 10 resources in it. And the root render pass has a
     // mask resource used when drawing the child render pass, as well as its
     // replica (it's added twice). The number 10 may change if
     // AppendOneOfEveryQuadType() is updated, and the value here should be
     // updated accordingly.
-    EXPECT_EQ(22u, last_frame.delegated_frame_data->resource_list.size());
+    EXPECT_EQ(22u, last_frame->delegated_frame_data->resource_list.size());
 
     EndTest();
   }