ozone: add NV12 to GetBufferFormatFromFourCCFormat

ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org

Committed: https://crrev.com/05e587d647ad21227c035870a67237d918c3e080
Cr-Commit-Position: refs/heads/master@{#437608}

[Daniele Castagna, 2016-12-09 02:35:29]

Thank you!
Just two nits. I think specifying just ui: or just ozone: in the title/first
line of the CL comment is enough.

Also, I think you meant to refer to b/33011213 not b:33011213 in the TEST= line.

[Daniele Castagna, 2016-12-09 02:37:32]

On 2016/12/09 at 02:35:29, Daniele Castagna wrote:
> Thank you!
> Just two nits. I think specifying just ui: or just ozone: in the title/first
line of the CL comment is enough.
> 
> Also, I think you meant to refer to b/33011213 not b:33011213 in the TEST=
line.

I'm probably wrong about b/ and I've been using the wrong format for a while
now.

LGTM

[gurchetansingh, 2016-12-09 04:49:46]

Description was changed from

==========
ui: ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org
==========

to

==========
ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org
==========

[gurchetansingh, 2016-12-09 04:50:32]

On 2016/12/09 02:35:29, Daniele Castagna wrote:
> Thank you!
> Just two nits. I think specifying just ui: or just ozone: in the title/first
> line of the CL comment is enough.
> 

Done

[dnicoara, 2016-12-09 17:43:28]

lgtm

[gurchetansingh, 2016-12-09 18:29:39]

The CQ bit was checked by gurchetansingh@chromium.org

[commit-bot: I haz the power, 2016-12-09 18:30:22]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-09 19:30:09]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1481308179035580, "parent_rev":
"d371aa8a6a661e31148ad3370114ea27111e825c", "commit_rev":
"b6f3d51849be022fe84293a5d7f28e497409b2b5"}

[commit-bot: I haz the power, 2016-12-09 19:30:36]

Description was changed from

==========
ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org
==========

to

==========
ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org

Review-Url: https://codereview.chromium.org/2562903002
==========

[commit-bot: I haz the power, 2016-12-09 19:30:37]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-12-12 14:36:54]

Description was changed from

==========
ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org

Review-Url: https://codereview.chromium.org/2562903002
==========

to

==========
ozone: add NV12 to GetBufferFormatFromFourCCFormat

crrev.com/2533163002 changed the internal format representation
of the GbmBuffer class. The function used to convert back to the
gfx::BufferFormat type, GetBufferFormatFromFourCCFormat, is missing
NV12. This format gets passed from the ARC++ container to Chrome, so
video playback isn't working on some platforms. Let's fix this by
adding the format.

BUG=b:33011213
TEST=Play Youtube App video on minnie

R=dnicoara@chromium.org, dcastagna@chromium.org, tfiga@chromium.org,
posciak@chromium.org

Committed: https://crrev.com/05e587d647ad21227c035870a67237d918c3e080
Cr-Commit-Position: refs/heads/master@{#437608}
==========

[commit-bot: I haz the power, 2016-12-12 14:36:54]

Patchset 1 (id:??) landed as
https://crrev.com/05e587d647ad21227c035870a67237d918c3e080
Cr-Commit-Position: refs/heads/master@{#437608}