ozone: Implement zero/one-copy texture for Ozone GBM.

content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.cc

 // Copyright 2014 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.
 
 #include "content/common/gpu/client/gpu_memory_buffer_impl_ozone_native_pixmap.h"

[reveman, 2015/08/06 12:08:35]

why do we need any of these changes to this file? I think it was much cleaner
before.

[dshwang, 2015/08/06 13:59:17]

Done.

 
 #include "ui/ozone/public/client_native_pixmap_factory.h"
-#include "ui/ozone/public/surface_factory_ozone.h"
 
 namespace content {
 
 GpuMemoryBufferImplOzoneNativePixmap::GpuMemoryBufferImplOzoneNativePixmap(
     gfx::GpuMemoryBufferId id,
     const gfx::Size& size,
     gfx::BufferFormat format,
+    gfx::BufferUsage usage,
     const DestructionCallback& callback,
     scoped_ptr<ui::ClientNativePixmap> pixmap)
-    : GpuMemoryBufferImpl(id, size, format, callback), pixmap_(pixmap.Pass()) {}
+    : GpuMemoryBufferImpl(id, size, format, callback),
+      usage_(usage),
+      pixmap_(pixmap.Pass()) {}
 
 GpuMemoryBufferImplOzoneNativePixmap::~GpuMemoryBufferImplOzoneNativePixmap() {}
 
 // static
 scoped_ptr<GpuMemoryBufferImpl>
 GpuMemoryBufferImplOzoneNativePixmap::CreateFromHandle(
     const gfx::GpuMemoryBufferHandle& handle,
     const gfx::Size& size,
     gfx::BufferFormat format,
     gfx::BufferUsage usage,
     const DestructionCallback& callback) {
-  scoped_ptr<ui::ClientNativePixmap> native_pixmap =
-      ui::ClientNativePixmapFactory::GetInstance()->ImportFromHandle(
-          handle.native_pixmap_handle, size, format, usage);
+  scoped_ptr<ui::ClientNativePixmap> native_pixmap;
+  switch (usage) {
+    case gfx::BufferUsage::MAP:
+      native_pixmap = ui::ClientNativePixmapFactory::GetInstance()
+                          ->ImportFromHandle(handle.native_pixmap_handle, size,
+                                             format, usage)
+                          .Pass();
+      if (!native_pixmap)
+        return nullptr;
+      break;
+    case gfx::BufferUsage::SCANOUT:
+      break;
+    case gfx::BufferUsage::PERSISTENT_MAP:
+      NOTREACHED();
+      return nullptr;
+  }
+
   return make_scoped_ptr<GpuMemoryBufferImpl>(
-      new GpuMemoryBufferImplOzoneNativePixmap(handle.id, size, format,
+      new GpuMemoryBufferImplOzoneNativePixmap(handle.id, size, format, usage,
                                                callback, native_pixmap.Pass()));
 }
 
 bool GpuMemoryBufferImplOzoneNativePixmap::Map(void** data) {
+  DCHECK(!mapped_);
+  DCHECK(usage_ == gfx::BufferUsage::MAP);
+  DCHECK(pixmap_);
+  mapped_ = true;
   return pixmap_->Map(data);
 }
 
 void GpuMemoryBufferImplOzoneNativePixmap::Unmap() {
+  DCHECK(mapped_);
+  DCHECK(usage_ == gfx::BufferUsage::MAP);
+  DCHECK(pixmap_);
   pixmap_->Unmap();
+  mapped_ = false;
 }
 
 void GpuMemoryBufferImplOzoneNativePixmap::GetStride(int* stride) const {
+  DCHECK(usage_ == gfx::BufferUsage::MAP);
+  DCHECK(pixmap_);
   pixmap_->GetStride(stride);
 }
 
 gfx::GpuMemoryBufferHandle GpuMemoryBufferImplOzoneNativePixmap::GetHandle()
     const {
   gfx::GpuMemoryBufferHandle handle;
   handle.type = gfx::OZONE_NATIVE_PIXMAP;
   handle.id = id_;
   return handle;
 }
 
 }  // namespace content