gl: Add ozone native pixmap bind test.

ui/gl/gl_image_ozone_native_pixmap_unittest.cc

-// Copyright 2015 The Chromium Authors. All rights reserved.
+// Copyright 2016 The Chromium Authors. All rights reserved.

[reveman, 2016/04/21 02:31:31]

nit: I don't think we update the year in chromium code

[Daniele Castagna, 2016/04/22 07:45:39]

Done.

 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdint.h>
 
 #include "testing/gtest/include/gtest/gtest.h"
 #include "ui/gfx/buffer_types.h"
 #include "ui/gl/gl_image_ozone_native_pixmap.h"
 #include "ui/gl/test/gl_image_test_template.h"
+#include "ui/ozone/public/client_native_pixmap.h"
+#include "ui/ozone/public/client_native_pixmap_factory.h"
 #include "ui/ozone/public/ozone_platform.h"
 #include "ui/ozone/public/surface_factory_ozone.h"
 
 namespace gl {
 namespace {
 
+template <gfx::BufferUsage usage>
 class GLImageOzoneNativePixmapTestDelegate {
  public:
   scoped_refptr<gl::GLImage> CreateSolidColorImage(
       const gfx::Size& size,
       const uint8_t color[4]) const {
     ui::SurfaceFactoryOzone* surface_factory =
         ui::OzonePlatform::GetInstance()->GetSurfaceFactoryOzone();
     scoped_refptr<ui::NativePixmap> pixmap =
         surface_factory->CreateNativePixmap(gfx::kNullAcceleratedWidget, size,
                                             gfx::BufferFormat::RGBA_8888,
-                                            gfx::BufferUsage::SCANOUT);
+                                            usage);
     EXPECT_TRUE(pixmap != nullptr);
+    if (usage == gfx::BufferUsage::GPU_READ_CPU_READ_WRITE) {
+      auto client_pixmap_factory = ui::ClientNativePixmapFactory::Create();

[reveman, 2016/04/21 02:31:31]

Would it make more sense to have the factory be a member of
GLImageOzoneNativePixmapTestDelegate? Looks a bit weird that we create a
factory, use it to create a pixmap and then destroy the factory before the
pixmap.

[Daniele Castagna, 2016/04/22 07:45:39]

Moved.

+      auto client_pixmap = client_pixmap_factory->ImportFromHandle(
+          pixmap->ExportHandle(), size, usage);
+      void* data = client_pixmap->Map();
+      EXPECT_TRUE(data);
+      GLImageTestSupport::SetBufferDataToColor(
+          size.width(), size.height(), pixmap->GetDmaBufPitch(), 0,
+          pixmap->GetBufferFormat(), color, static_cast<uint8_t*>(data));
+      client_pixmap->Unmap();
+    }
+
     scoped_refptr<gfx::GLImageOzoneNativePixmap> image(
         new gfx::GLImageOzoneNativePixmap(size, GL_RGBA));
     EXPECT_TRUE(image->Initialize(pixmap.get(), pixmap->GetBufferFormat()));
     return image;
   }
 
   unsigned GetTextureTarget() const { return GL_TEXTURE_2D; }
 };
 
+INSTANTIATE_TYPED_TEST_CASE_P(
+    GLImageOzoneNativePixmap,
+    GLImageTest,
+    GLImageOzoneNativePixmapTestDelegate<gfx::BufferUsage::SCANOUT>);
+
 INSTANTIATE_TYPED_TEST_CASE_P(GLImageOzoneNativePixmap,
-                              GLImageTest,
-                              GLImageOzoneNativePixmapTestDelegate);
+                              GLImageBindTest,
+                              GLImageOzoneNativePixmapTestDelegate<
+                                  gfx::BufferUsage::GPU_READ_CPU_READ_WRITE>);
 
 }  // namespace
 }  // namespace gl