Revert of Make cc::Display not own its BeginFrameSource

Revert of Make cc::Display not own its BeginFrameSource (patchset #5 id:80001 of https://codereview.chromium.org/2286273003/ )

Reason for revert:
Suspect this may have induced renderer crashes in the context_lost tests; see http://crbug.com/642984 .

Original issue's description:
> Make cc::Display not own its BeginFrameSource
>
> This is refactoring to allow MusBrowserCompositorOutputSurface to
> provide an ExternalBeginFrameSource that it owns rather than
> passing ownership to Display.
>
> This should also make it possible to remove begin frame sources from
> OutputSurface in the future after the OutputSurface/CompositorFrameSink
> split.
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/9848a61393772cc8a9aa8349c68f7d436c743369
> Cr-Commit-Position: refs/heads/master@{#415738}

TBR=boliu@chromium.org,danakj@chromium.org,fsamuel@chromium.org,nyquist@chromium.org,sievers@chromium.org,enne@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/97ecf8ee67f2ce4a10bdd11c99c5fd83f2da8934
Cr-Commit-Position: refs/heads/master@{#415813}

[Ken Russell (switch to Gerrit), 2016-09-01 00:08:02]

The CQ bit was checked by kbr@chromium.org

[Ken Russell (switch to Gerrit), 2016-09-01 00:08:03]

Created Revert of Make cc::Display not own its BeginFrameSource

[commit-bot: I haz the power, 2016-09-01 00:08:31]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-01 00:10:22]

Description was changed from

==========
Revert of Make cc::Display not own its BeginFrameSource (patchset #5 id:80001 of
https://codereview.chromium.org/2286273003/ )

Reason for revert:
Suspect this may have induced renderer crashes in the context_lost tests; see
http://crbug.com/642984 .


Original issue's description:
> Make cc::Display not own its BeginFrameSource
> 
> This is refactoring to allow MusBrowserCompositorOutputSurface to
> provide an ExternalBeginFrameSource that it owns rather than
> passing ownership to Display.
> 
> This should also make it possible to remove begin frame sources from
> OutputSurface in the future after the OutputSurface/CompositorFrameSink
> split.
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/9848a61393772cc8a9aa8349c68f7d436c743369
> Cr-Commit-Position: refs/heads/master@{#415738}

TBR=boliu@chromium.org,danakj@chromium.org,fsamuel@chromium.org,nyquist@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Make cc::Display not own its BeginFrameSource (patchset #5 id:80001 of
https://codereview.chromium.org/2286273003/ )

Reason for revert:
Suspect this may have induced renderer crashes in the context_lost tests; see
http://crbug.com/642984 .


Original issue's description:
> Make cc::Display not own its BeginFrameSource
> 
> This is refactoring to allow MusBrowserCompositorOutputSurface to
> provide an ExternalBeginFrameSource that it owns rather than
> passing ownership to Display.
> 
> This should also make it possible to remove begin frame sources from
> OutputSurface in the future after the OutputSurface/CompositorFrameSink
> split.
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/9848a61393772cc8a9aa8349c68f7d436c743369
> Cr-Commit-Position: refs/heads/master@{#415738}

TBR=boliu@chromium.org,danakj@chromium.org,fsamuel@chromium.org,nyquist@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[commit-bot: I haz the power, 2016-09-01 00:10:24]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-01 00:13:32]

Description was changed from

==========
Revert of Make cc::Display not own its BeginFrameSource (patchset #5 id:80001 of
https://codereview.chromium.org/2286273003/ )

Reason for revert:
Suspect this may have induced renderer crashes in the context_lost tests; see
http://crbug.com/642984 .


Original issue's description:
> Make cc::Display not own its BeginFrameSource
> 
> This is refactoring to allow MusBrowserCompositorOutputSurface to
> provide an ExternalBeginFrameSource that it owns rather than
> passing ownership to Display.
> 
> This should also make it possible to remove begin frame sources from
> OutputSurface in the future after the OutputSurface/CompositorFrameSink
> split.
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
> 
> Committed: https://crrev.com/9848a61393772cc8a9aa8349c68f7d436c743369
> Cr-Commit-Position: refs/heads/master@{#415738}

TBR=boliu@chromium.org,danakj@chromium.org,fsamuel@chromium.org,nyquist@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Make cc::Display not own its BeginFrameSource (patchset #5 id:80001 of
https://codereview.chromium.org/2286273003/ )

Reason for revert:
Suspect this may have induced renderer crashes in the context_lost tests; see
http://crbug.com/642984 .

Original issue's description:
> Make cc::Display not own its BeginFrameSource
>
> This is refactoring to allow MusBrowserCompositorOutputSurface to
> provide an ExternalBeginFrameSource that it owns rather than
> passing ownership to Display.
>
> This should also make it possible to remove begin frame sources from
> OutputSurface in the future after the OutputSurface/CompositorFrameSink
> split.
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_precise_blink_rel
>
> Committed: https://crrev.com/9848a61393772cc8a9aa8349c68f7d436c743369
> Cr-Commit-Position: refs/heads/master@{#415738}

TBR=boliu@chromium.org,danakj@chromium.org,fsamuel@chromium.org,nyquist@chrom...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

Committed: https://crrev.com/97ecf8ee67f2ce4a10bdd11c99c5fd83f2da8934
Cr-Commit-Position: refs/heads/master@{#415813}
==========

[commit-bot: I haz the power, 2016-09-01 00:13:34]

Patchset 1 (id:??) landed as
https://crrev.com/97ecf8ee67f2ce4a10bdd11c99c5fd83f2da8934
Cr-Commit-Position: refs/heads/master@{#415813}