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

cc/test/fake_output_surface.h

Index: cc/test/fake_output_surface.h
diff --git a/cc/test/fake_output_surface.h b/cc/test/fake_output_surface.h
index 7c5add8aed990676449550e6eb5a65e7a7e0144b..39ff003e6701256947e46bc1a0f3cc203b9ff507 100644
--- a/cc/test/fake_output_surface.h
+++ b/cc/test/fake_output_surface.h
@@ -109,10 +109,10 @@ class FakeOutputSurface : public OutputSurface {
     capabilities_.max_frames_pending = max;
   }
 
-  CompositorFrame& last_sent_frame() { return last_sent_frame_; }
+  CompositorFrame* last_sent_frame() { return last_sent_frame_.get(); }
   size_t num_sent_frames() { return num_sent_frames_; }
 
-  void SwapBuffers(CompositorFrame* frame) override;
+  void SwapBuffers(std::unique_ptr<CompositorFrame> frame) override;
 
   OutputSurfaceClient* client() { return client_; }
   bool BindToClient(OutputSurfaceClient* client) override;
@@ -170,7 +170,7 @@ class FakeOutputSurface : public OutputSurface {
                     bool delegated_rendering);
 
   OutputSurfaceClient* client_ = nullptr;
-  CompositorFrame last_sent_frame_;
+  std::unique_ptr<CompositorFrame> last_sent_frame_;

[danakj, 2016/06/23 20:34:41]

even if you make CompositorFrame moveonly and passed around by value, using a
unique_ptr here is nice to denote the "no frame sent yet" state, i like the
changes to tests that came from this.

   size_t num_sent_frames_ = 0;
   bool has_external_stencil_test_ = false;
   bool suspended_for_recycle_ = false;