Clear pending reads after suspend completes, not before.

Clear pending reads after suspend completes, not before.

If reads are cleared prior to suspend that does not guarantee that
no further read has been started (even when playback rate == 0) since
the renderers may be prerolling buffers.

The abort before suspend actually does nothing but cause additional
churn in some cases. Instead we should destroy the renderer and then
abort any stale state.

BUG=638018
Committed: https://crrev.com/c672f0c53548b3250ca5e4573f3aa6452fe6ba82
Cr-Commit-Position: refs/heads/master@{#419941}

[DaleCurtis, 2016-09-21 00:23:02]

dalecurtis@chromium.org changed reviewers:
+ sandersd@chromium.org

[sandersd (OOO until July 31), 2016-09-21 00:26:09]

lgtm

[DaleCurtis, 2016-09-21 00:26:54]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2016-09-21 00:27:29]

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-21 01:58:34]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-09-21 02:02:13]

Description was changed from

==========
Clear pending reads after suspend completes, not before.

If reads are cleared prior to suspend that does not guarantee that
no further read has been started (even when playback rate == 0) since
the renderers may be prerolling buffers.

The abort before suspend actually does nothing but cause additional
churn in some cases. Instead we should destroy the renderer and then
abort any stale state.

BUG=638018
==========

to

==========
Clear pending reads after suspend completes, not before.

If reads are cleared prior to suspend that does not guarantee that
no further read has been started (even when playback rate == 0) since
the renderers may be prerolling buffers.

The abort before suspend actually does nothing but cause additional
churn in some cases. Instead we should destroy the renderer and then
abort any stale state.

BUG=638018

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

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

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