[Reland]Remove cc::Display::begin_frame_source_

cc::Display shouldn't know about BeginFrameSource. Remove begin_frame_source_
and move related work to where cc::Display is created/initialized.

DirectCompositorFrameSink and GpuRootCompositorFrameSink registers Display's
BeginFrameSource before initializing the Display. MusDisplayProvider and
GpuProcessTransportFactory sets BeginFrameSource to DisplayScheduler before
passing the scheduler to the Display.

In CompositorImpl, the BeginFrameSource is registered after a cc::Display is
created in InitializeDisplay and unregistered right before the Display is
destroyed in SetVisibility.

Bug: 730710, 722935
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I723bc91a13f9013925ebb2f91e87dc698f4f5bd3
Reviewed-on: https://chromium-review.googlesource.com/527478
Commit-Queue: Xingyu Zhang <staraz@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Reviewed-by: danakj <danakj@chromium.org>
Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#478389}

• android_webview/browser/surfaces_instance.cc[diff]
• cc/surfaces/direct_compositor_frame_sink_unittest.cc[diff]
• cc/surfaces/display.cc[diff]
• cc/surfaces/display.h[diff]
• cc/surfaces/display_scheduler.cc[diff]
• cc/surfaces/display_scheduler.h[diff]
• cc/surfaces/display_scheduler_unittest.cc[diff]
• cc/surfaces/display_unittest.cc[diff]
• cc/test/test_compositor_frame_sink.cc[diff]
• components/viz/display_compositor/gpu_display_provider.cc[diff]
• components/viz/frame_sinks/gpu_root_compositor_frame_sink.cc[diff]
• content/browser/compositor/gpu_process_transport_factory.cc[diff]
• content/browser/renderer_host/compositor_impl_android.cc[diff]
• content/renderer/android/synchronous_compositor_frame_sink.cc[diff]
• ui/compositor/test/in_process_context_factory.cc[diff]