Always post mailbox release in GpuVideoDecoder; avoids deadlock.

Always post mailbox release in GpuVideoDecoder; avoids deadlock.

We can get into a situation where GVD::ReleaseMailbox() attempts to
grab the GL context lock while the compositor holds it wheil waiting
on a lock in VideoRendererImpl which may also be held during a call
to GVD::ReleaseMailbox().

Not sure how to test this since we have no test harness capable of
instantiating all these components at the same time and some live
across content/ and media/ boundaries.

BUG=710209
TEST=none

Review-Url: https://codereview.chromium.org/2812783002
Cr-Commit-Position: refs/heads/master@{#463857}
Committed: https://chromium.googlesource.com/chromium/src/+/0a9f673ccd4f82b350c06df622f323b255ebab21

[DaleCurtis, 2017-04-10 22:53:29]

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

[sandersd (OOO until July 31), 2017-04-11 22:27:10]

lgtm

[DaleCurtis, 2017-04-11 22:38:21]

The CQ bit was checked by dalecurtis@chromium.org

[commit-bot: I haz the power, 2017-04-11 22:39:33]

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-04-12 00:36:19]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491950301935490, "parent_rev":
"cf65b91bc7496b3dd2c958c890984e73c26ad22c", "commit_rev":
"0a9f673ccd4f82b350c06df622f323b255ebab21"}

[commit-bot: I haz the power, 2017-04-12 00:37:13]

Description was changed from

==========
Always post mailbox release in GpuVideoDecoder; avoids deadlock.

We can get into a situation where GVD::ReleaseMailbox() attempts to
grab the GL context lock while the compositor holds it wheil waiting
on a lock in VideoRendererImpl which may also be held during a call
to GVD::ReleaseMailbox().

Not sure how to test this since we have no test harness capable of
instantiating all these components at the same time and some live
across content/ and media/ boundaries.

BUG=710209
TEST=none
==========

to

==========
Always post mailbox release in GpuVideoDecoder; avoids deadlock.

We can get into a situation where GVD::ReleaseMailbox() attempts to
grab the GL context lock while the compositor holds it wheil waiting
on a lock in VideoRendererImpl which may also be held during a call
to GVD::ReleaseMailbox().

Not sure how to test this since we have no test harness capable of
instantiating all these components at the same time and some live
across content/ and media/ boundaries.

BUG=710209
TEST=none

Review-Url: https://codereview.chromium.org/2812783002
Cr-Commit-Position: refs/heads/master@{#463857}
Committed:
https://chromium.googlesource.com/chromium/src/+/0a9f673ccd4f82b350c06df622f3...
==========

[commit-bot: I haz the power, 2017-04-12 00:37:14]

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