exo: Add support for wl_drm version 2 to wayland bindings.

exo: Add support for wl_drm version 2 to wayland bindings.

BUG=549781
TEST=exo_unittests --gtest_filter=DisplayTest.CreatePrimeBuffer

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

[reveman, 2015-11-01 20:36:46]

Description was changed from

==========
exo: Add support for wl_drm version 2 to wayland bindings.

TEST=TEST=exo_unittests --gtest_filter=DisplayTest.CreatePrimeBuffer
==========

to

==========
exo: Add support for wl_drm version 2 to wayland bindings.

TEST=exo_unittests --gtest_filter=DisplayTest.CreatePrimeBuffer
==========

[reveman, 2015-11-20 05:36:22]

reveman@chromium.org changed reviewers:
+ piman@chromium.org

[reveman, 2015-11-20 05:36:22]

This depends on https://codereview.chromium.org/1423563007.

[piman, 2015-11-20 19:40:34]

lgtm

https://codereview.chromium.org/1427743004/diff/100001/components/exo/display...
File components/exo/display_unittest.cc (right):

https://codereview.chromium.org/1427743004/diff/100001/components/exo/display...
components/exo/display_unittest.cc:71:
base::ScopedFD(native_pixmap_handle.fd.fd), buffer_size, drm_format,
drm_format? Doesn't CreatePrimeBuffer take a gfx::BufferFormat instead?

https://codereview.chromium.org/1427743004/diff/100001/components/exo/wayland...
File components/exo/wayland/server.cc (right):

https://codereview.chromium.org/1427743004/diff/100001/components/exo/wayland...
components/exo/wayland/server.cc:430: const auto supported_format =
nit: const auto*

[reveman, 2015-11-20 20:53:52]

The CQ bit was checked by reveman@chromium.org

[reveman, 2015-11-20 20:53:53]

https://codereview.chromium.org/1427743004/diff/100001/components/exo/display...
File components/exo/display_unittest.cc (right):

https://codereview.chromium.org/1427743004/diff/100001/components/exo/display...
components/exo/display_unittest.cc:71:
base::ScopedFD(native_pixmap_handle.fd.fd), buffer_size, drm_format,
On 2015/11/20 at 19:40:34, piman (slow to review) wrote:
> drm_format? Doesn't CreatePrimeBuffer take a gfx::BufferFormat instead?

Done. Forgot to update this when changing this from a previous patch..

https://codereview.chromium.org/1427743004/diff/100001/components/exo/wayland...
File components/exo/wayland/server.cc (right):

https://codereview.chromium.org/1427743004/diff/100001/components/exo/wayland...
components/exo/wayland/server.cc:430: const auto supported_format =
On 2015/11/20 at 19:40:34, piman (slow to review) wrote:
> nit: const auto*

Done.

[reveman, 2015-11-20 20:53:53]

The patchset sent to the CQ was uploaded after l-g-t-m from piman@chromium.org
Link to the patchset: https://codereview.chromium.org/1427743004/#ps120001
(title: "fix unit test and nit")

[commit-bot: I haz the power, 2015-11-20 20:54:57]

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

[commit-bot: I haz the power, 2015-11-20 21:50:22]

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

[commit-bot: I haz the power, 2015-11-20 21:50:23]

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

[reveman, 2015-11-20 22:08:32]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2015-11-20 22:09:54]

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

[commit-bot: I haz the power, 2015-11-20 23:38:32]

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

[commit-bot: I haz the power, 2015-11-20 23:38:33]

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

[reveman, 2015-11-20 23:39:08]

The CQ bit was checked by reveman@chromium.org

[commit-bot: I haz the power, 2015-11-20 23:41:03]

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

[commit-bot: I haz the power, 2015-11-21 02:46:38]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-11-21 02:47:34]

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