Browser side OZONE_NATIVE_BUFFER allocation

content/common/gpu/client/gpu_memory_buffer_factory_host.h

Index: content/common/gpu/client/gpu_memory_buffer_factory_host.h
diff --git a/content/common/gpu/client/gpu_memory_buffer_factory_host.h b/content/common/gpu/client/gpu_memory_buffer_factory_host.h
index 6dfd58beabaa88f55ab0926f2b26353cb4eb02ba..89d42c986bfd4022d84bc2337a0b54b6a54b96a4 100644
--- a/content/common/gpu/client/gpu_memory_buffer_factory_host.h
+++ b/content/common/gpu/client/gpu_memory_buffer_factory_host.h
@@ -6,6 +6,7 @@
 #define CONTENT_COMMON_GPU_CLIENT_GPU_MEMORY_BUFFER_FACTORY_HOST_H_
 
 #include "base/callback.h"
+#include "content/common/content_export.h"
 
 namespace gfx {
 class Size;
@@ -28,8 +29,11 @@ class CONTENT_EXPORT GpuMemoryBufferFactoryHost {
   virtual void DestroyGpuMemoryBuffer(const gfx::GpuMemoryBufferHandle& handle,
                                       int32 sync_point) = 0;
 
+  static GpuMemoryBufferFactoryHost* GetInstance();

[reveman, 2014/08/13 17:00:25]

nit: I'd move this static method above all virtual member functions.

[reveman, 2014/08/13 17:00:25]

Don't we need a InitInstance or SetInstance too? and where is the implementation
of this?

[achaulk, 2014/08/13 17:10:12]

Forgot to add the new file. I'm currently just doing the initialization in the
ctor, which matches how BrowserGpuChannelHostFactory does it

+
  protected:
-  virtual ~GpuMemoryBufferFactoryHost() {}
+  GpuMemoryBufferFactoryHost();
+  virtual ~GpuMemoryBufferFactoryHost();

[reveman, 2014/08/13 17:00:26]

hm, why do you need to change this?

 };
 
 }  // namespace content