Cancel the starvation callback when MediaSourcePlayer::Release() gets called.

Cancel the starvation callback when MediaSourcePlayer::Release() gets called.

If we don't cancel the starvation callback, ProcessPendingEvents will get executed later when the callback fires.
Because we use a BarrierClosure in ProcessPendingEvents() and the count is 0 as both audio and video decoders are released, OnPrefetchDone() will be immediately called.
And this will cause ProcessPendingEvents() to get called again and again, thus crashing chrome.

An alternative to this fix is to check count before creating the barrier.
Logically starvation callback should not get called after Release().

BUG=285573
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=221509

[qinmin, 2013-09-05 03:04:00]

PTAL

[acolwell GONE FROM CHROMIUM, 2013-09-05 14:40:30]

LGTM

[commit-bot: I haz the power, 2013-09-05 16:10:55]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/qinmin@chromium.org/23672025/1

[xhwang, 2013-09-05 16:17:20]

lgtm

[wolenetz, 2013-09-05 16:30:40]

lgtm

[wolenetz, 2013-09-05 16:30:43]

lgtm

[commit-bot: I haz the power, 2013-09-05 20:44:03]

Change committed as 221509