VAVDA: properly mark outputted pictures.

VAVDA: properly mark outputted pictures.

For some very mysterious reason the flag flip that should have been marking
H264Pictures as outputted on OutputPic has vanished. This CL restores it.

BUG=142451
TEST=by hand



Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=151854

[marcheu, 2012-08-14 03:59:06]

LGTM

Makes me wonder how it worked before. I guess removebypoc used to do the
cleanup, so we'd hang onto the pictures for longer than needed.

[Pawel Osciak, 2012-08-14 05:02:17]

On 2012/08/14 03:59:06, marcheu wrote:
> LGTM
> 
> Makes me wonder how it worked before. I guess removebypoc used to do the
> cleanup, so we'd hang onto the pictures for longer than needed.

This could have potentially made us (attempt to) display one buffer twice. It
couldn't cause leaks per se, but would cause two GPU jobs to operate on one
buffer (one after another though), the second getting there after it was
released after the first one.
I could not reproduce this on Debug builds at all, and since 99% of my testing
had been on Debug, I hadn't seen it before.

[commit-bot: I haz the power, 2012-08-14 05:06:36]

No LGTM from a valid reviewer yet. Only full committers are accepted.
Even if an LGTM may have been provided, it was from a non-committer or
a lowly provisional committer, _not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[piman, 2012-08-14 06:11:07]

lgtm

[Ami GONE FROM CHROMIUM, 2012-08-14 06:13:31]

lgtm

[commit-bot: I haz the power, 2012-08-14 06:13:34]

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

[commit-bot: I haz the power, 2012-08-14 10:31:12]

Try job failure for 10855142-1 (retry) on linux_chromeos for steps
"browser_tests, content_browsertests".
It's a second try, previously, steps "browser_tests, content_browsertests"
failed.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...

[commit-bot: I haz the power, 2012-08-14 11:56:54]

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

[commit-bot: I haz the power, 2012-08-16 02:04:56]

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

[commit-bot: I haz the power, 2012-08-16 04:37:02]

Change committed as 151854