gpu: Have GpuService create and own GpuMemoryBufferFactory.

gpu/ipc/service/gpu_memory_buffer_factory.cc

Index: gpu/ipc/service/gpu_memory_buffer_factory.cc
diff --git a/gpu/ipc/service/gpu_memory_buffer_factory.cc b/gpu/ipc/service/gpu_memory_buffer_factory.cc
index d3dbb554a7e3e4824003d6cd774393a17643f386..ed016cd46c48e462b1537278833f190950a4b5d3 100644
--- a/gpu/ipc/service/gpu_memory_buffer_factory.cc
+++ b/gpu/ipc/service/gpu_memory_buffer_factory.cc
@@ -7,6 +7,7 @@
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "build/build_config.h"
+#include "gpu/ipc/common/gpu_memory_buffer_support.h"
 
 #if defined(OS_MACOSX)
 #include "gpu/ipc/service/gpu_memory_buffer_factory_io_surface.h"
@@ -21,6 +22,8 @@ namespace gpu {
 // static
 std::unique_ptr<GpuMemoryBufferFactory>
 GpuMemoryBufferFactory::CreateNativeType() {
+  if (GetNativeGpuMemoryBufferType() == gfx::EMPTY_BUFFER)

[reveman, 2017/03/31 06:13:34]

nit: can we just remove the NOTREACHED below instead? I think all devices that
will return non-null will have native type != gfx::EMPTY_BUFFER and even if they
didn't creating the factory if possible shouldn't hurt.

[sadrul, 2017/03/31 13:09:15]

Done.

+    return nullptr;
 #if defined(OS_MACOSX)
   return base::WrapUnique(new GpuMemoryBufferFactoryIOSurface);
 #endif