sync compositor: Signal futures on channel shutdown

sync compositor: Signal futures on channel shutdown

If the channel shutdown (cleanly or on error) while there are still
pending frame futures in the filter, then browser may wait indefinitely
on those futures. In this case, signal those futures with empty frames.
What frame gets drawn in these cases do not matter very much.

Also add a |filter_ready_| state so that new futures inserted gets
signaled immediately.

BUG=636164
Committed: https://crrev.com/3e3ad9a95aa746107d49fd120811e686b915e861
Cr-Commit-Position: refs/heads/master@{#427742}

[boliu, 2016-10-26 15:46:39]

boliu@chromium.org changed reviewers:
+ sgurun@chromium.org

[boliu, 2016-10-26 15:46:40]

ptal

[sgurun-gerrit only, 2016-10-26 17:23:07]

lgtm

[boliu, 2016-10-26 17:25:40]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2016-10-26 17:26:19]

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-10-26 18:07:22]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-10-26 18:21:49]

Description was changed from

==========
sync compositor: Signal futures on channel shutdown

If the channel shutdown (cleanly or on error) while there are still
pending frame futures in the filter, then browser may wait indefinitely
on those futures. In this case, signal those futures with empty frames.
What frame gets drawn in these cases do not matter very much.

Also add a |filter_ready_| state so that new futures inserted gets
signaled immediately.

BUG=636164
==========

to

==========
sync compositor: Signal futures on channel shutdown

If the channel shutdown (cleanly or on error) while there are still
pending frame futures in the filter, then browser may wait indefinitely
on those futures. In this case, signal those futures with empty frames.
What frame gets drawn in these cases do not matter very much.

Also add a |filter_ready_| state so that new futures inserted gets
signaled immediately.

BUG=636164

Committed: https://crrev.com/3e3ad9a95aa746107d49fd120811e686b915e861
Cr-Commit-Position: refs/heads/master@{#427742}
==========

[commit-bot: I haz the power, 2016-10-26 18:21:49]

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