gl: Test YV12 buffer drm import.

gl: Test YV12 buffer drm import.

Adds a test to check that importing YV12 format created via drm works
and that it is possible to draw after creating an EGL image backed by
them.

BUG=618516
TEST=gl_unittests on samus.

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

[Daniele Castagna, 2016-06-08 22:08:59]

dcastagna@chromium.org changed reviewers:
+ dnicoara@chromium.org, posciak@chromium.org, reveman@chromium.org

[Daniele Castagna, 2016-06-08 22:08:59]

dnicoara@, this is just the code for the drm import test.

I agree with you with what you said in crrev.com/2052543004
but I think it could be still useful to land this while we work
on extending minigbm and Ozone to support YV12.
WDYT?

[dnicoara, 2016-06-09 00:45:18]

On 2016/06/08 22:08:59, Daniele Castagna wrote:
> dnicoara@, this is just the code for the drm import test.
> 
> I agree with you with what you said in crrev.com/2052543004
> but I think it could be still useful to land this while we work
> on extending minigbm and Ozone to support YV12.
> WDYT?

Could this at least be tracked in a bug?

[Daniele Castagna, 2016-06-09 00:51:30]

On 2016/06/09 at 00:45:18, dnicoara wrote:
> On 2016/06/08 22:08:59, Daniele Castagna wrote:
> > dnicoara@, this is just the code for the drm import test.
> > 
> > I agree with you with what you said in crrev.com/2052543004
> > but I think it could be still useful to land this while we work
> > on extending minigbm and Ozone to support YV12.
> > WDYT?
> 
> Could this at least be tracked in a bug?

Absolutely: crbug.com/618516

[Daniele Castagna, 2016-06-09 00:51:40]

Description was changed from

==========
gl: Test YV12 buffer drm import.

Adds a test to check that importing YV12 format created via drm works
and that it is possible to draw after creating an EGL image backed by
them.

TEST=gl_unittests on samus.
==========

to

==========
gl: Test YV12 buffer drm import.

Adds a test to check that importing YV12 format created via drm works
and that it is possible to draw after creating an EGL image backed by
them.

BUG=618516
TEST=gl_unittests on samus.
==========

[dnicoara, 2016-06-09 00:55:22]

Thank you! lgtm

[reveman, 2016-06-09 01:08:47]

lgtm

[Daniele Castagna, 2016-06-09 01:29:02]

dcastagna@chromium.org changed reviewers:
+ ccameron@chromium.org

[Daniele Castagna, 2016-06-09 01:29:02]

+Chris for owner approval.

[ccameron, 2016-06-09 06:22:43]

OWNER rs lgtm

[Daniele Castagna, 2016-06-09 16:23:24]

The CQ bit was checked by dcastagna@chromium.org

[Daniele Castagna, 2016-06-09 16:23:24]

The patchset sent to the CQ was uploaded after l-g-t-m from
dnicoara@chromium.org, reveman@chromium.org, ccameron@chromium.org
Link to the patchset: https://codereview.chromium.org/2052543004/#ps40001
(title: "Change patch dependency")

[commit-bot: I haz the power, 2016-06-09 16:24:06]

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

[commit-bot: I haz the power, 2016-06-09 17:26:17]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-06-09 17:26:25]

CQ bit was unchecked

[commit-bot: I haz the power, 2016-06-09 17:30:23]

Description was changed from

==========
gl: Test YV12 buffer drm import.

Adds a test to check that importing YV12 format created via drm works
and that it is possible to draw after creating an EGL image backed by
them.

BUG=618516
TEST=gl_unittests on samus.
==========

to

==========
gl: Test YV12 buffer drm import.

Adds a test to check that importing YV12 format created via drm works
and that it is possible to draw after creating an EGL image backed by
them.

BUG=618516
TEST=gl_unittests on samus.

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

[commit-bot: I haz the power, 2016-06-09 17:30:25]

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