gpu: Have GpuService create and own GpuMemoryBufferFactory.

gpu/ipc/service/gpu_memory_buffer_factory.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 "gpu/ipc/service/gpu_memory_buffer_factory.h"
 
 #include "base/logging.h"
 #include "base/memory/ptr_util.h"
 #include "build/build_config.h"
+#include "gpu/ipc/common/gpu_memory_buffer_support.h"

[reveman, 2017/03/31 13:38:38]

no need for this after removing the lines below:

[sadrul, 2017/04/02 02:50:50]

Done.

 
 #if defined(OS_MACOSX)
 #include "gpu/ipc/service/gpu_memory_buffer_factory_io_surface.h"
 #endif
 
 #if defined(USE_OZONE)
 #include "gpu/ipc/service/gpu_memory_buffer_factory_ozone_native_pixmap.h"
 #endif
 
 namespace gpu {
 
 // static
 std::unique_ptr<GpuMemoryBufferFactory>
 GpuMemoryBufferFactory::CreateNativeType() {
+  if (GetNativeGpuMemoryBufferType() == gfx::EMPTY_BUFFER)
+    return nullptr;

[reveman, 2017/03/31 13:38:38]

remove these two lines now that it's fine to return null below

[sadrul, 2017/04/02 02:50:50]

Done. Also, updated the doc to say that this can return null when native buffers
are not supported.

 #if defined(OS_MACOSX)
   return base::WrapUnique(new GpuMemoryBufferFactoryIOSurface);
 #endif
 #if defined(USE_OZONE)
   return base::WrapUnique(new GpuMemoryBufferFactoryOzoneNativePixmap);
 #endif
-  NOTREACHED();
   return nullptr;
 }
 
 }  // namespace gpu