[Video] Reset new processed frame callback on the compositor thread.

[Video] Reset new processed frame callback on the compositor thread.

The callback is set and used on the compositor thread but is reset on the
media thread, causing a data race.
Moved resetting the callback to the OnRenderingStateChange when the state is
set to false.

BUG=684458
TEST=manual

Review-Url: https://codereview.chromium.org/2651323002
Cr-Commit-Position: refs/heads/master@{#446515}
Committed: https://chromium.googlesource.com/chromium/src/+/b5c8d0d1bed4056880ad70ed949714f9edc0f81b

[whywhat, 2017-01-26 06:39:40]

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

[whywhat, 2017-01-26 06:39:41]

PTaL

[whywhat, 2017-01-26 06:39:44]

The CQ bit was checked by avayvod@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-26 06:40:00]

Dry run: 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-01-26 08:13:30]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-26 08:13:31]

Dry run: This issue passed the CQ dry run.

[DaleCurtis, 2017-01-26 17:32:03]

Whoops, sorry I missed this in earlier review, lgtm

[whywhat, 2017-01-27 00:57:53]

The CQ bit was checked by avayvod@chromium.org

[commit-bot: I haz the power, 2017-01-27 00:59:27]

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-01-27 01:11:40]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485478673250940, "parent_rev":
"c80c4a6801f438c230a65c6d81d65457efc1e431", "commit_rev":
"b5c8d0d1bed4056880ad70ed949714f9edc0f81b"}

[commit-bot: I haz the power, 2017-01-27 01:12:59]

Description was changed from

==========
[Video] Reset new processed frame callback on the compositor thread.

The callback is set and used on the compositor thread but is reset on the
media thread, causing a data race.
Moved resetting the callback to the OnRenderingStateChange when the state is
set to false.

BUG=684458
TEST=manual
==========

to

==========
[Video] Reset new processed frame callback on the compositor thread.

The callback is set and used on the compositor thread but is reset on the
media thread, causing a data race.
Moved resetting the callback to the OnRenderingStateChange when the state is
set to false.

BUG=684458
TEST=manual

Review-Url: https://codereview.chromium.org/2651323002
Cr-Commit-Position: refs/heads/master@{#446515}
Committed:
https://chromium.googlesource.com/chromium/src/+/b5c8d0d1bed4056880ad70ed9497...
==========

[commit-bot: I haz the power, 2017-01-27 01:13:00]

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