Revert of cc::SurfaceDependencyTracker should not crash when a Display goes away

Revert of cc::SurfaceDependencyTracker should not crash when a Display goes away (patchset #16 id:300001 of https://codereview.chromium.org/2880023002/ )

Reason for revert:
I think this is probably responsible for new flake like this:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_ChromiumOS_MSan_Tests%2F739%2F%2B%2Frecipes%2Fsteps%2Fbrowser_tests%2F0%2Flogs%2FSamlTest.MetaRefreshToHTTPDisallowed%2F0

Original issue's description:
> cc::SurfaceDependencyTracker should not crash when a Display goes away
> 
> Previously, SurfaceDependencyTracker was a BeginFrameObserver that 
> directly observed BeginFrames from the first display. This is a problem 
> if the first display goes away. This CL solves this problem by making
> SurfaceDependencyTracker observe an independent 
> "PrimaryBeginFrameSource" instead. MojoFrameSinkManager owns 
> SurfaceDependencyTracker and the PrimaryBeginFrameSource. When
> SurfaceDependencyTracker needs BeginFrames, that request goes to
> PrimaryBeginFrameSource which then adds itself as a BeginFrameObserver 
> to the primary BeginFrameSource.
> 
> When the primary BeginFrameSource goes away, PrimaryBeginFrameSource 
> removes itself as an observer and, if necessary, adds itself as an 
> observer arbitrarily to another BeginFrameSource. Future BeginFrames 
> get forwarded from the PrimaryBeginFrameSource to 
> SurfaceDependencyTracker.
> 
> BUG=672962
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2880023002
> Cr-Commit-Position: refs/heads/master@{#474539}
> Committed: https://chromium.googlesource.com/chromium/src/+/b520968f4ac40fb89ce25329134b388ca42cca12

TBR=enne@chromium.org,vmpstr@chromium.org,fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672962
Review-Url: https://codereview.chromium.org/2908473002
Cr-Commit-Position: refs/heads/master@{#474653}
Committed: https://chromium.googlesource.com/chromium/src/+/a94c5212e856eec73332d8c761dfa905199bd064

[Ken Rockot(use gerrit already), 2017-05-25 15:01:33]

The CQ bit was checked by rockot@chromium.org

[Ken Rockot(use gerrit already), 2017-05-25 15:01:33]

Created Revert of cc::SurfaceDependencyTracker should not crash when a Display
goes away

[commit-bot: I haz the power, 2017-05-25 15:01:55]

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, 2017-05-25 15:02:31]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495724493001880, "parent_rev":
"b29b8d8833bf2f8fa7c1a33a90cf7d96a44d79ef", "commit_rev":
"a94c5212e856eec73332d8c761dfa905199bd064"}

[commit-bot: I haz the power, 2017-05-25 15:02:46]

Description was changed from

==========
Revert of cc::SurfaceDependencyTracker should not crash when a Display goes away
(patchset #16 id:300001 of https://codereview.chromium.org/2880023002/ )

Reason for revert:
I think this is probably responsible for new flake like this:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_...

Original issue's description:
> cc::SurfaceDependencyTracker should not crash when a Display goes away
> 
> Previously, SurfaceDependencyTracker was a BeginFrameObserver that 
> directly observed BeginFrames from the first display. This is a problem 
> if the first display goes away. This CL solves this problem by making
> SurfaceDependencyTracker observe an independent 
> "PrimaryBeginFrameSource" instead. MojoFrameSinkManager owns 
> SurfaceDependencyTracker and the PrimaryBeginFrameSource. When
> SurfaceDependencyTracker needs BeginFrames, that request goes to
> PrimaryBeginFrameSource which then adds itself as a BeginFrameObserver 
> to the primary BeginFrameSource.
> 
> When the primary BeginFrameSource goes away, PrimaryBeginFrameSource 
> removes itself as an observer and, if necessary, adds itself as an 
> observer arbitrarily to another BeginFrameSource. Future BeginFrames 
> get forwarded from the PrimaryBeginFrameSource to 
> SurfaceDependencyTracker.
> 
> BUG=672962
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2880023002
> Cr-Commit-Position: refs/heads/master@{#474539}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b520968f4ac40fb89ce25329134b...

TBR=enne@chromium.org,vmpstr@chromium.org,fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672962
==========

to

==========
Revert of cc::SurfaceDependencyTracker should not crash when a Display goes away
(patchset #16 id:300001 of https://codereview.chromium.org/2880023002/ )

Reason for revert:
I think this is probably responsible for new flake like this:

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.memory%2FLinux_...

Original issue's description:
> cc::SurfaceDependencyTracker should not crash when a Display goes away
> 
> Previously, SurfaceDependencyTracker was a BeginFrameObserver that 
> directly observed BeginFrames from the first display. This is a problem 
> if the first display goes away. This CL solves this problem by making
> SurfaceDependencyTracker observe an independent 
> "PrimaryBeginFrameSource" instead. MojoFrameSinkManager owns 
> SurfaceDependencyTracker and the PrimaryBeginFrameSource. When
> SurfaceDependencyTracker needs BeginFrames, that request goes to
> PrimaryBeginFrameSource which then adds itself as a BeginFrameObserver 
> to the primary BeginFrameSource.
> 
> When the primary BeginFrameSource goes away, PrimaryBeginFrameSource 
> removes itself as an observer and, if necessary, adds itself as an 
> observer arbitrarily to another BeginFrameSource. Future BeginFrames 
> get forwarded from the PrimaryBeginFrameSource to 
> SurfaceDependencyTracker.
> 
> BUG=672962
> CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
> 
> Review-Url: https://codereview.chromium.org/2880023002
> Cr-Commit-Position: refs/heads/master@{#474539}
> Committed:
https://chromium.googlesource.com/chromium/src/+/b520968f4ac40fb89ce25329134b...

TBR=enne@chromium.org,vmpstr@chromium.org,fsamuel@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672962

Review-Url: https://codereview.chromium.org/2908473002
Cr-Commit-Position: refs/heads/master@{#474653}
Committed:
https://chromium.googlesource.com/chromium/src/+/a94c5212e856eec73332d8c761df...
==========

[commit-bot: I haz the power, 2017-05-25 15:02:48]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/a94c5212e856eec73332d8c761df...