cc: Fix accelerated video freezing with commit aborts

cc: Fix accelerated video freezing with commit aborts

Although VideoLayer doesn't update any resources, it needs to not let
commits abort when it has been invalidated as this is currently the
mechanism on desktop to tell the compositor to redraw it with a new
frame.

R=danakj@chromium.org
BUG=266350
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=215169

[danakj, 2013-08-01 20:22:01]

LGTM

test?

https://codereview.chromium.org/21567003/diff/1/cc/layers/video_layer.cc
File cc/layers/video_layer.cc (right):

https://codereview.chromium.org/21567003/diff/1/cc/layers/video_layer.cc#newc...
cc/layers/video_layer.cc:30: // TODO(enne): Have the video frame provider
communicate this directly
While this will be good, I don't think we'll want to delete this line once we
have that WMP impl working on the compositor thread only. So I am not sure about
this todo. It kinda belongs in WMPClientImpl.

[danakj, 2013-08-01 20:22:54]

https://codereview.chromium.org/21567003/diff/1/cc/layers/video_layer.cc
File cc/layers/video_layer.cc (right):

https://codereview.chromium.org/21567003/diff/1/cc/layers/video_layer.cc#newc...
cc/layers/video_layer.cc:30: // TODO(enne): Have the video frame provider
communicate this directly
On 2013/08/01 20:22:01, danakj wrote:
> While this will be good, I don't think we'll want to delete this line once we
> have that WMP impl working on the compositor thread only. So I am not sure
about
> this todo. It kinda belongs in WMPClientImpl.

I guess we could make this a DCHECK, but there is a 3rd WMP impl at least still.

[enne (OOO), 2013-08-01 21:02:15]

Added a test, removed TODO and left a better comment.

[danakj, 2013-08-01 21:04:51]

LGTM! Thanks!

[commit-bot: I haz the power, 2013-08-01 21:07:46]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/enne@chromium.org/21567003/7001

[commit-bot: I haz the power, 2013-08-02 00:36:43]

Change committed as 215169