Add media::VideoFrame::WrapNativeTextures()

Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in addition to 1-mailbox and 3-mailbox frames. It replaces every usage of WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/b97bf4cfe9d8666c0f48f5b0487c722648e6cd1c
Cr-Commit-Position: refs/heads/master@{#393116}

[jbauman, 2016-05-06 01:11:47]

Description was changed from

==========
Add WrapNativeTextures.

Handles both WrapNativeTexture and WrapYUV.
==========

to

==========
Add WrapNativeTextures.

Handles both WrapNativeTexture and WrapYUV.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbauman, 2016-05-07 00:13:53]

Description was changed from

==========
Add WrapNativeTextures.

Handles both WrapNativeTexture and WrapYUV.
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in
addition to 1-mailbox and 3-mailbox frames. It replaces every usage of
WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[jbauman, 2016-05-07 00:28:24]

jbauman@chromium.org changed reviewers:
+ hubbe@chromium.org, miu@chromium.org, piman@chromium.org

[jbauman, 2016-05-07 00:28:25]

miu@: please review
hubbe@: media/ OWNERS review
piman@: cc/ OWNERS review

[piman, 2016-05-09 16:27:16]

lgtm

[miu, 2016-05-10 00:40:07]

lgtm. Much nicer! :-)

[jbauman, 2016-05-11 22:09:43]

hubbe@: ping

[hubbe, 2016-05-11 22:20:14]

lgtm

[jbauman, 2016-05-11 22:48:37]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2016-05-11 22:48:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960563002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960563002/40001

[commit-bot: I haz the power, 2016-05-11 22:57:15]

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

[commit-bot: I haz the power, 2016-05-11 22:57:16]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[jbauman, 2016-05-11 22:59:13]

jbauman@chromium.org changed reviewers:
+ alokp@chromium.org

[jbauman, 2016-05-11 22:59:14]

alokp: chromecast/ OWNERS review.

[alokp, 2016-05-11 23:02:25]

lgtm

[jbauman, 2016-05-11 23:29:06]

The CQ bit was checked by jbauman@chromium.org

[commit-bot: I haz the power, 2016-05-11 23:29:58]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1960563002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1960563002/40001

[commit-bot: I haz the power, 2016-05-12 00:20:34]

Description was changed from

==========
Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in
addition to 1-mailbox and 3-mailbox frames. It replaces every usage of
WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in
addition to 1-mailbox and 3-mailbox frames. It replaces every usage of
WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[commit-bot: I haz the power, 2016-05-12 00:20:36]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-05-12 00:23:46]

Description was changed from

==========
Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in
addition to 1-mailbox and 3-mailbox frames. It replaces every usage of
WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
Add media::VideoFrame::WrapNativeTextures()

This is more flexible, as it lets us specify NV12 frames with 2 mailboxes, in
addition to 1-mailbox and 3-mailbox frames. It replaces every usage of
WrapNativeTexture() and WrapYUV420NativeTextures().

BUG=574292

CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/b97bf4cfe9d8666c0f48f5b0487c722648e6cd1c
Cr-Commit-Position: refs/heads/master@{#393116}
==========

[commit-bot: I haz the power, 2016-05-12 00:23:47]

Patchset 3 (id:??) landed as
https://crrev.com/b97bf4cfe9d8666c0f48f5b0487c722648e6cd1c
Cr-Commit-Position: refs/heads/master@{#393116}

[ccameron, 2016-05-16 17:19:28]

On 2016/05/12 00:23:47, commit-bot: I haz the power wrote:
> Patchset 3 (id:??) landed as
> https://crrev.com/b97bf4cfe9d8666c0f48f5b0487c722648e6cd1c
> Cr-Commit-Position: refs/heads/master@{#393116}

This has changed the way that YUV textures are drawn on Mac (and possibly other
platforms). Trying to figure out a fix, if I can't soon, I'll have to revert.