ozone: rename to GpuMemoryBuffer(Factory|Impl)OzoneNativePixmap.

ozone: rename to GpuMemoryBuffer(Factory|Impl)OzoneNativePixmap.

GpuMemoryBuffer(Factory|Impl)OzoneNativeBuffer eventually creates NativePixmap,
so rename them to GpuMemoryBuffer(Factory|Impl)OzoneNativePixmap.

BUG=475633
Committed: https://crrev.com/6a80cc0d45f2f51dd4604ef2f78473e43786eb6c
Cr-Commit-Position: refs/heads/master@{#340258}

[dshwang, 2015-07-20 17:22:06]

dongseong.hwang@intel.com changed reviewers:
+ alexst@chromium.org, reveman@chromium.org, spang@chromium.org,
tiago.vignatti@intel.com

[dshwang, 2015-07-20 17:22:07]

reveman, could you review?

reveman request to rename NativePixmap to NativeBuffer in
https://codereview.chromium.org/1134993003/#msg16
However, the term NativePixmap is widely used in ozone implementation, so rename
NativeBuffer to NativePixmap.

spang, could you review ui/ozone?

[spang, 2015-07-20 18:12:13]

lgtm

[reveman, 2015-07-20 20:07:55]

lgtm

[dshwang, 2015-07-21 10:05:10]

dongseong.hwang@intel.com changed reviewers:
+ nasko@chromium.org

[dshwang, 2015-07-21 10:05:11]

thank you for reviewing.

nasko, could you review content/browser/gpu, and content/common/gpu/client?

[nasko, 2015-07-21 11:27:25]

There are more specific OWNERs for GPU code (content/common/gpu/OWNERS,
content/browser/gpu/OWNERS). Please use those, as I have very little
understanding of this code.

[nasko, 2015-07-21 11:27:33]

nasko@chromium.org changed reviewers:
- nasko@chromium.org

[dshwang, 2015-07-21 11:47:32]

dongseong.hwang@intel.com changed reviewers:
+ sievers@chromium.org

[dshwang, 2015-07-21 11:48:15]

sievers, could you review content/browser/gpu, and content/common/gpu/client?

[dshwang, 2015-07-22 14:48:56]

On 2015/07/21 11:48:15, dshwang wrote:
> sievers, could you review content/browser/gpu, and content/common/gpu/client?

sievers, could you review this renaming CL?

[reveman, 2015-07-22 19:47:39]

Please change the name of gfx::GpuMemoryBufferType::OZONE_NATIVE_BUFFER to
gfx::GpuMemoryBufferType::OZONE_NATIVE_PIXMAP as part of this change.

[dshwang, 2015-07-23 08:26:55]

On 2015/07/22 19:47:39, reveman wrote:
> Please change the name of gfx::GpuMemoryBufferType::OZONE_NATIVE_BUFFER to
> gfx::GpuMemoryBufferType::OZONE_NATIVE_PIXMAP as part of this change.

Good point. Done.

sievers, could you review this renaming CL?

[no sievers, 2015-07-23 18:10:39]

lgtm

https://codereview.chromium.org/1240353002/diff/20001/content/common/gpu/clie...
File content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.h
(right):

https://codereview.chromium.org/1240353002/diff/20001/content/common/gpu/clie...
content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.h:12: //
Implementation of GPU memory buffer based on Ozone native buffers.
nit: 'based on Ozone native pixmaps'?

[dshwang, 2015-07-24 10:34:07]

Thank you for reviewing! addressed nits.

https://codereview.chromium.org/1240353002/diff/20001/content/common/gpu/clie...
File content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.h
(right):

https://codereview.chromium.org/1240353002/diff/20001/content/common/gpu/clie...
content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.h:12: //
Implementation of GPU memory buffer based on Ozone native buffers.
On 2015/07/23 18:10:39, sievers wrote:
> nit: 'based on Ozone native pixmaps'?

Done.

[dshwang, 2015-07-24 10:34:13]

The CQ bit was checked by dongseong.hwang@intel.com

[dshwang, 2015-07-24 10:34:13]

The patchset sent to the CQ was uploaded after l-g-t-m from
reveman@chromium.org, spang@chromium.org, sievers@chromium.org
Link to the patchset: https://codereview.chromium.org/1240353002/#ps40001
(title: "address nits in comments")

[commit-bot: I haz the power, 2015-07-24 10:34:52]

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

[commit-bot: I haz the power, 2015-07-24 11:55:19]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-07-24 11:55:55]

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