Make cc::Display not own its BeginFrameSource

cc/test/test_delegating_output_surface.h

Index: cc/test/test_delegating_output_surface.h
diff --git a/cc/test/test_delegating_output_surface.h b/cc/test/test_delegating_output_surface.h
index 2ec1e413e6d8767bd9a7dd83c187179705a213a0..edce55102a06c38a9eb205461d5fa1634f03187f 100644
--- a/cc/test/test_delegating_output_surface.h
+++ b/cc/test/test_delegating_output_surface.h
@@ -88,6 +88,8 @@ class TestDelegatingOutputSurface : public OutputSurface,
   std::unique_ptr<SurfaceIdAllocator> surface_id_allocator_;
   SurfaceId delegated_surface_id_;
 
+  std::unique_ptr<SyntheticBeginFrameSource> begin_frame_source_;

[danakj, 2016/08/30 01:10:56]

Can you put this right on top of display_ since that's all that uses it?

+
   // Uses surface_manager_.
   std::unique_ptr<SurfaceFactory> surface_factory_;