Always post buffering state updates since they may change sink state.

Always post buffering state updates since they may change sink state.

The lock must never be held when OnTimeStateChanged() is calling into
the VideoRendererSink or we can deadlock the compositor thread.

With video-only playback occurs a buffering state change for underflow
would immediately invoke OnTimeStateChanged(false) which would try to
stop the VideoRendererSink under lock.

BUG=496466
TEST=video only playback doesn't deadlock.

Committed: https://crrev.com/9d8a53f3bf01aed28abbd1f9a713a25362c0d158
Cr-Commit-Position: refs/heads/master@{#332772}

[DaleCurtis, 2015-06-04 00:53:20]

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

[DaleCurtis, 2015-06-04 00:53:21]

No test since it's across component boundaries and we can't DCHECK() that a lock
_is not_ acquired in a function :|

[xhwang, 2015-06-04 00:59:34]

lgtm

[DaleCurtis, 2015-06-04 01:00:57]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2015-06-04 01:01:10]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1161463006/1

[commit-bot: I haz the power, 2015-06-04 03:01:45]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-06-04 03:03:41]

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

[Yuta Kitamura, 2015-06-04 09:41:39]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1159573005/ by yutak@chromium.org.

The reason for reverting is: Speculative revert for Blink layout tests crashes
on Nexus 4:
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Ne...
(SEGV on CrGpuMain thread is observed)

Sorry if my guess is not correct; there wasn't
a clear signal to identify the culprit..

[Yuta Kitamura, 2015-06-04 10:45:44]

On 2015/06/04 09:41:39, Yuta Kitamura wrote:
> A revert of this CL (patchset #1 id:1) has been created in
> https://codereview.chromium.org/1159573005/ by mailto:yutak@chromium.org.
> 
> The reason for reverting is: Speculative revert for Blink layout tests crashes
> on Nexus 4:
>
http://build.chromium.org/p/chromium.webkit/builders/WebKit%20Android%20%28Ne...
> (SEGV on CrGpuMain thread is observed)
> 
> Sorry if my guess is not correct; there wasn't
> a clear signal to identify the culprit..

Sorry for the hassle, it turned out the your patch was
innocent, and I'm landing a revert of the revert.