Make cc::Display not own its BeginFrameSource

cc/surfaces/display.h

Index: cc/surfaces/display.h
diff --git a/cc/surfaces/display.h b/cc/surfaces/display.h
index bf786ae15c48bf839311097c02ce2bf894b17fd4..a558184ac1182988cb660d0a53dac81169952768 100644
--- a/cc/surfaces/display.h
+++ b/cc/surfaces/display.h
@@ -57,7 +57,7 @@ class CC_SURFACES_EXPORT Display : public DisplaySchedulerClient,
   Display(SharedBitmapManager* bitmap_manager,
           gpu::GpuMemoryBufferManager* gpu_memory_buffer_manager,
           const RendererSettings& settings,
-          std::unique_ptr<BeginFrameSource> begin_frame_source,
+          BeginFrameSource* begin_frame_source,
           std::unique_ptr<OutputSurface> output_surface,
           std::unique_ptr<DisplayScheduler> scheduler,
           std::unique_ptr<TextureMailboxDeleter> texture_mailbox_deleter);
@@ -131,9 +131,9 @@ class CC_SURFACES_EXPORT Display : public DisplaySchedulerClient,
   gfx::Rect external_viewport_;
   bool output_is_secure_ = false;
 
-  // The begin_frame_source_ is often known by the output_surface_ and
-  // the scheduler_.
-  std::unique_ptr<BeginFrameSource> begin_frame_source_;
+  // The begin_frame_source_ is often also known by the output_surface_ and the
+  // scheduler_.
+  BeginFrameSource* begin_frame_source_;

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

can it be a const member maybe up with the bitmap_manager_ etc

   std::unique_ptr<OutputSurface> output_surface_;
   std::unique_ptr<DisplayScheduler> scheduler_;
   std::unique_ptr<ResourceProvider> resource_provider_;