Move native_pixmap.h from ui/ozone/public to ui/gfx

Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2688623004
Cr-Commit-Position: refs/heads/master@{#454585}
Committed: https://chromium.googlesource.com/chromium/src/+/eacee6c8bf6fe847bf6b5b08a260badd7ed64930

[Julien Isorce, 2017-02-09 12:10:50]

Description was changed from

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
==========

to

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Julien Isorce, 2017-02-09 12:11:16]

The CQ bit was checked by julien.isorce@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-09 12:11:31]

Dry run: 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, 2017-02-09 15:43:37]

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

[commit-bot: I haz the power, 2017-02-09 15:43:38]

Dry run: This issue passed the CQ dry run.

[Julien Isorce, 2017-02-21 16:59:39]

On 2017/02/09 15:43:38, commit-bot: I haz the power wrote:
> Dry run: This issue passed the CQ dry run.

PTAL.

[reveman, 2017-02-23 03:22:59]

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

[reveman, 2017-02-23 03:22:59]

lgtm % dnicoara's review

[Julien Isorce, 2017-02-23 13:08:57]

The CQ bit was checked by julien.isorce@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-23 13:09:06]

Dry run: 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, 2017-02-23 15:00:03]

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

[commit-bot: I haz the power, 2017-02-23 15:00:04]

Dry run: This issue passed the CQ dry run.

[dnicoara, 2017-02-23 16:02:38]

dnicoara@chromium.org changed reviewers:
+ rjkroege@chromium.org

[dnicoara, 2017-02-23 16:02:39]

lgtm

+ rjkroege@ for broader Ozone ownership review

[Julien Isorce, 2017-02-24 10:25:10]

Description was changed from

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

[Julien Isorce, 2017-02-24 10:25:11]

julien.isorce@chromium.org changed reviewers:
+ sandersd@chromium.org

[Julien Isorce, 2017-02-24 10:29:44]

On 2017/02/23 16:02:39, dnicoara wrote:
> lgtm
> 
> + rjkroege@ for broader Ozone ownership review

And + sandersd@ for media/gpu. Please take a look. Thx.

[sandersd (OOO until July 31), 2017-03-02 00:37:29]

Sorry I missed this one, media/ LGTM.

[Julien Isorce, 2017-03-02 15:28:26]

On 2017/03/02 00:37:29, sandersd (OOO until March 13) wrote:
> Sorry I missed this one, media/ LGTM.

No worries, thx for the reviews.

[Julien Isorce, 2017-03-02 15:29:14]

Hi rjkroege@ please take a look, thx.

[rjkroege, 2017-03-02 15:37:39]

lgtm

[Julien Isorce, 2017-03-03 11:26:42]

The CQ bit was checked by julien.isorce@chromium.org

[Julien Isorce, 2017-03-03 11:26:43]

The patchset sent to the CQ was uploaded after l-g-t-m from
reveman@chromium.org, rjkroege@chromium.org, dnicoara@chromium.org,
sandersd@chromium.org
Link to the patchset: https://codereview.chromium.org/2688623004/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2017-03-03 11:26:55]

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, 2017-03-03 11:32:58]

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

[commit-bot: I haz the power, 2017-03-03 11:32:59]

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

[Julien Isorce, 2017-03-03 11:39:51]

julien.isorce@chromium.org changed reviewers:
+ sadrul@chromium.org

[Julien Isorce, 2017-03-03 11:40:08]

On 2017/03/03 11:32:59, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
>
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

** Presubmit ERRORS **
Missing LGTM from an OWNER for these files:
    ui/gfx/native_pixmap.h

Hi sadrul@, please take a look. Thx!

[sadrul, 2017-03-03 15:01:08]

lgtm

[Julien Isorce, 2017-03-03 15:08:17]

The CQ bit was checked by julien.isorce@chromium.org

[commit-bot: I haz the power, 2017-03-03 15:08:31]

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, 2017-03-03 15:12:14]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1488553697668170,
"parent_rev": "6d2b4f8955d1733666dfc7a0d0309b65566fa20d", "commit_rev":
"eacee6c8bf6fe847bf6b5b08a260badd7ed64930"}

[commit-bot: I haz the power, 2017-03-03 15:12:54]

Description was changed from

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
==========

to

==========
Move native_pixmap.h from ui/ozone/public to ui/gfx

Like existing ui/gfx/native_pixmap_handle.h, the file native_pixmap.h
is not ozone specific.

Pre-step before moving ui/ozone/gl/gl_image_ozone_native_pixmap.cc to
ui/gl/gl_image_linux_native_pixmap.cc in order to enable gl::GLImageEGL
with EGL_LINUX_DMA_BUF_EXT on linux desktop.

BUG=584248
R=reveman@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel

Review-Url: https://codereview.chromium.org/2688623004
Cr-Commit-Position: refs/heads/master@{#454585}
Committed:
https://chromium.googlesource.com/chromium/src/+/eacee6c8bf6fe847bf6b5b08a260...
==========

[commit-bot: I haz the power, 2017-03-03 15:12:55]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/eacee6c8bf6fe847bf6b5b08a260...