Browser side OZONE_NATIVE_BUFFER allocation

content/common/gpu/client/gpu_memory_buffer_impl_linux.cc

 // Copyright 2013 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.h"
 
+#include "content/common/gpu/client/gpu_memory_buffer_impl_ozone.h"
 #include "content/common/gpu/client/gpu_memory_buffer_impl_shared_memory.h"
 
 namespace content {
+#if defined(USE_OZONE)

[reveman, 2014/08/11 19:43:05]

No idefs in these files. Please add a platform specific file for ozone instead
of using _linux.cc. It should be named gpu_memory_buffer_impl_ozone.cc.

[achaulk, 2014/08/12 19:03:56]

Ok

+namespace {
+const int kBrowserClientId = 1;
+}  // namespace
+#endif  // defined(USE_OZONE)
 
 // static
 scoped_ptr<GpuMemoryBufferImpl> GpuMemoryBufferImpl::Create(
     const gfx::Size& size,
     unsigned internalformat,
     unsigned usage) {
+#if defined(USE_OZONE)
+  if (GpuMemoryBufferImplOzone::IsConfigurationSupported(
+          size, internalformat, usage)) {
+    scoped_ptr<GpuMemoryBufferImplOzone> buffer(
+        new GpuMemoryBufferImplOzone(size, internalformat, kBrowserClientId));
+    if (!buffer->Initialize())
+      return scoped_ptr<GpuMemoryBufferImpl>();
+
+    return buffer.PassAs<GpuMemoryBufferImpl>();
+  }
+#endif  // defined(USE_OZONE)

[reveman, 2014/08/11 19:43:05]

do we need browser compositor support for this?

[achaulk, 2014/08/12 19:03:57]

Not for using them. We will eventually to render the main scene into an overlay.
They get allocated via CreateImageChromium and there's already support for that

[reveman, 2014/08/12 20:19:11]

I'm failing to see how this works. Latest patch is just generating an id by
assuming that the client id for the browser is 1 but not telling the GPU process
to allocate any storage. Can you explain a bit more how this is supposed to
work? 

[achaulk, 2014/08/12 21:38:01]

That is something that we will need to figure out later. I'll remove it for now
though since it can be included in another patch where it will be used.

+
   if (GpuMemoryBufferImplSharedMemory::IsConfigurationSupported(
           size, internalformat, usage)) {
     scoped_ptr<GpuMemoryBufferImplSharedMemory> buffer(
         new GpuMemoryBufferImplSharedMemory(size, internalformat));
     if (!buffer->Initialize())
       return scoped_ptr<GpuMemoryBufferImpl>();
 
     return buffer.PassAs<GpuMemoryBufferImpl>();
   }
 
   return scoped_ptr<GpuMemoryBufferImpl>();
 }
 
 // static
 void GpuMemoryBufferImpl::AllocateForChildProcess(
     const gfx::Size& size,
     unsigned internalformat,
     unsigned usage,
     base::ProcessHandle child_process,
+    int client_id,
+    GpuMemoryBufferFactoryHost* gpu_factory_host,
     const AllocationCallback& callback) {
+#if defined(USE_OZONE)
+  if (GpuMemoryBufferImplOzone::IsConfigurationSupported(
+          size, internalformat, usage)) {
+    GpuMemoryBufferImplOzone::AllocateSharedMemoryForChildProcess(
+        size,
+        internalformat,
+        child_process,
+        client_id,
+        gpu_factory_host,
+        callback);
+    return;
+  }
+#endif  // defined(USE_OZONE)
   if (GpuMemoryBufferImplSharedMemory::IsConfigurationSupported(
           size, internalformat, usage)) {
     GpuMemoryBufferImplSharedMemory::AllocateSharedMemoryForChildProcess(
         size, internalformat, child_process, callback);
     return;
   }
 
   callback.Run(gfx::GpuMemoryBufferHandle());
 }
 
@@ skipping 11 matching lines @@
     unsigned internalformat) {
   switch (handle.type) {
     case gfx::SHARED_MEMORY_BUFFER: {
       scoped_ptr<GpuMemoryBufferImplSharedMemory> buffer(
           new GpuMemoryBufferImplSharedMemory(size, internalformat));
       if (!buffer->InitializeFromHandle(handle))
         return scoped_ptr<GpuMemoryBufferImpl>();
 
       return buffer.PassAs<GpuMemoryBufferImpl>();
     }
+#if defined(USE_OZONE)
+    case gfx::OZONE_NATIVE_BUFFER: {
+      scoped_ptr<GpuMemoryBufferImplOzone> buffer(
+          new GpuMemoryBufferImplOzone(size, internalformat, 0));
+      if (!buffer->InitializeFromHandle(handle))
+        return scoped_ptr<GpuMemoryBufferImpl>();
+
+      return buffer.PassAs<GpuMemoryBufferImpl>();
+    }
+#endif  // defined(USE_OZONE)
     default:
       return scoped_ptr<GpuMemoryBufferImpl>();
   }
 }
 
 }  // namespace content