Add format modifier IDs for EGL_EXT_image_dma_buf_import extension

ui/gfx/native_pixmap_handle_ozone.h

Index: ui/gfx/native_pixmap_handle_ozone.h
diff --git a/ui/gfx/native_pixmap_handle_ozone.h b/ui/gfx/native_pixmap_handle_ozone.h
index 3672feca1cab3bac6d9e7dd2676353ff488e821d..92b8bc5daffd5b572fe4c65ebd8adb1049d55857 100644
--- a/ui/gfx/native_pixmap_handle_ozone.h
+++ b/ui/gfx/native_pixmap_handle_ozone.h
@@ -13,6 +13,24 @@
 
 namespace gfx {
 
+// GbmBufferPlane is used to carry the plane related information for GBM
+// buffer. More fields can be added if they are plane specific.
+struct GFX_EXPORT GbmBufferPlane {
+  GbmBufferPlane();
+  GbmBufferPlane(int stride, int offset, uint64_t modifier);
+  GbmBufferPlane(const GbmBufferPlane& other);
+  ~GbmBufferPlane();
+
+  // The strides and offsets in bytes to be used when accessing the buffers via
+  // a memory mapping. One per plane per entry.
+  int stride;
+  int offset;
+  // The modifiers is retrieved from GBM library and passed to EGL driver.

[Daniele Castagna, 2016/06/15 17:30:09]

nit: s/modifiers/modifier or make the verb plural.

[vinceh, 2016/06/16 02:08:44]

Oops. Will fix.

+  // Generally it's platform specific, and we don't need to modify it in
+  // browser code. Also one per plane per entry.

[Daniele Castagna, 2016/06/15 17:30:09]

nit: if you use the term browser in chromium, we probably think about the
browser process and not about the whole thing. Maybe you can refer to it as
Chromium code instead?

[vinceh, 2016/06/16 02:08:44]

Yes, will fix.

+  uint64_t modifier;
+};
+
 struct GFX_EXPORT NativePixmapHandle {
   NativePixmapHandle();
   NativePixmapHandle(const NativePixmapHandle& other);
@@ -21,9 +39,7 @@ struct GFX_EXPORT NativePixmapHandle {
   // File descriptors for the underlying memory objects (usually dmabufs).
   std::vector<base::FileDescriptor> fds;
 
-  // The strides and offsets in bytes to be used when accessing the buffers via
-  // a memory mapping. One per plane per entry.
-  std::vector<std::pair<int, int>> strides_and_offsets;
+  std::vector<GbmBufferPlane> planes;
 };
 
 }  // namespace gfx