Fix DelegatedOutputSurface in Blimp

Fix DelegatedOutputSurface in Blimp

Make sure to build a BeginFrameSource when binding to the
OutputSurfaceClient.  Without this the compositor doesn't push frames.

This was broken by https://codereview.chromium.org/2083423006 because
BlimpLayerTreeSettings sets use_output_surface_begin_frame_source but
one wasn't provided by the DelegatedOutputSurface.

Infrastructure to add a Blimp test for this is almost complete and will
be added in a follow up CL!
BUG=

Committed: https://crrev.com/e5cc6b1bd2e6ab945027f79a9439966bec3ece38
Cr-Commit-Position: refs/heads/master@{#416395}

[David Trainor- moved to gerrit, 2016-09-02 22:54:08]

dtrainor@chromium.org changed reviewers:
+ khushalsagar@chromium.org

[David Trainor- moved to gerrit, 2016-09-02 22:54:08]

ptal thanks!

[Khushal, 2016-09-02 22:55:26]

lgtm

[David Trainor- moved to gerrit, 2016-09-02 23:09:48]

The CQ bit was checked by dtrainor@chromium.org

[commit-bot: I haz the power, 2016-09-02 23:10:14]

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-02 23:48:21]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-02 23:51:02]

Description was changed from

==========
Fix DelegatedOutputSurface in Blimp

Make sure to build a BeginFrameSource when binding to the
OutputSurfaceClient.  Without this the compositor doesn't push frames.

This was broken by https://codereview.chromium.org/2083423006 because
BlimpLayerTreeSettings sets use_output_surface_begin_frame_source but
one wasn't provided by the DelegatedOutputSurface.

Infrastructure to add a Blimp test for this is almost complete and will
be added in a follow up CL!
BUG=
==========

to

==========
Fix DelegatedOutputSurface in Blimp

Make sure to build a BeginFrameSource when binding to the
OutputSurfaceClient.  Without this the compositor doesn't push frames.

This was broken by https://codereview.chromium.org/2083423006 because
BlimpLayerTreeSettings sets use_output_surface_begin_frame_source but
one wasn't provided by the DelegatedOutputSurface.

Infrastructure to add a Blimp test for this is almost complete and will
be added in a follow up CL!
BUG=

Committed: https://crrev.com/e5cc6b1bd2e6ab945027f79a9439966bec3ece38
Cr-Commit-Position: refs/heads/master@{#416395}
==========

[commit-bot: I haz the power, 2016-09-02 23:51:03]

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

[Khushal, 2016-09-03 16:04:32]

https://codereview.chromium.org/2308143002/diff/1/blimp/client/core/composito...
File blimp/client/core/compositor/delegated_output_surface.cc (right):

https://codereview.chromium.org/2308143002/diff/1/blimp/client/core/composito...
blimp/client/core/compositor/delegated_output_surface.cc:56:
client->SetBeginFrameSource(begin_frame_source_.get());
Woops. Also destroy the |begin_frame_source_| and clean up the reference in
DetachFromClient?