Add format modifier IDs for EGL_EXT_image_dma_buf_import extension

ui/gfx/native_pixmap_handle_ozone.h

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef UI_GFX_NATIVE_PIXMAP_HANDLE_OZONE_H_
 #define UI_GFX_NATIVE_PIXMAP_HANDLE_OZONE_H_
 
 #include <stdint.h>
 #include <vector>
 
 #include "base/file_descriptor_posix.h"
 #include "ui/gfx/gfx_export.h"
 
 namespace gfx {
 
+struct GFX_EXPORT GbmBufferPlane {

[Daniele Castagna, 2016/06/15 03:50:20]

Comment before struct definition.

[vinceh, 2016/06/15 04:35:18]

Will fix.

+  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;
+  uint64_t modifier;

[Daniele Castagna, 2016/06/15 03:50:20]

nit: I think the modifier deserves a comment as well.

[vinceh, 2016/06/15 04:35:18]

Yes, will add.

+};
+
 struct GFX_EXPORT NativePixmapHandle {
   NativePixmapHandle();
   NativePixmapHandle(const NativePixmapHandle& other);
 
   ~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
 
 #endif  // UI_GFX_NATIVE_PIXMAP_HANDLE_OZONE_H_