cc: Recycle resources for software video decode.

cc: Recycle resources for software video decode.

When we update a compositor resource from a media::VideoFrame,
try to reuse resources from previous frames that were returned
to us by the ResourceProvider.

R=piman,enne
BUG=179729
Depends on: https://codereview.chromium.org/13445009/

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

[piman, 2013-04-10 00:56:30]

LGTM I guess, but I'll mention the thing I think is weird - that I mentioned on
IRC

https://codereview.chromium.org/13959005/diff/1/cc/resources/video_resource_u...
File cc/resources/video_resource_updater.cc (right):

https://codereview.chromium.org/13959005/diff/1/cc/resources/video_resource_u...
cc/resources/video_resource_updater.cc:359: GLC(context,
context->consumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name));
This looks weird, because it associates a mailbox with the texture ID in that
resource, that the RP doesn't know about - so if it tried to PrepareSendToParent
it, it would fail because we can't have 2 mailboxes associated with it. I gues
we won't try to PrepareSendToParent this resource but another one, but it's
still bending somewhat unexpectedly the expectations here.

[danakj, 2013-04-10 01:15:14]

https://codereview.chromium.org/13959005/diff/1/cc/resources/video_resource_u...
File cc/resources/video_resource_updater.cc (right):

https://codereview.chromium.org/13959005/diff/1/cc/resources/video_resource_u...
cc/resources/video_resource_updater.cc:359: GLC(context,
context->consumeTextureCHROMIUM(GL_TEXTURE_2D, mailbox.name));
On 2013/04/10 00:56:30, piman wrote:
> This looks weird, because it associates a mailbox with the texture ID in that
> resource, that the RP doesn't know about - so if it tried to
PrepareSendToParent
> it, it would fail because we can't have 2 mailboxes associated with it. I gues
> we won't try to PrepareSendToParent this resource but another one, but it's
> still bending somewhat unexpectedly the expectations here.

Thanks, I will add a TODO to remove ResourceProvider from here when we have
another way to create software/hardware backings that the RP can consume as
well.

[commit-bot: I haz the power, 2013-04-10 04:57:06]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/13959005/6001

[commit-bot: I haz the power, 2013-04-10 15:43:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/danakj@chromium.org/13959005/27001

[commit-bot: I haz the power, 2013-04-10 17:32:23]

Change committed as 193409