Rename ClientNativePixmapFactoryGbm to ClientNativePixmapFactoryDmabuf amd move to ui/gfx

ui/gfx/BUILD.gn

 # 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.
 
+import("//build/config/linux/pkg_config.gni")
 import("//build/config/ui.gni")
 import("//testing/test.gni")
 import("//testing/libfuzzer/fuzzer_test.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
 
+if (is_linux) {
+  pkg_config("libdrm") {
+    packages = [ "libdrm" ]

[spang, 2017/03/15 15:40:32]

+phajdan

Is linking this unconditionally into the linux desktop build OK? Or do we need
to use dlopen?

+  }
+}
+
 # Several targets want to include this header file, and some of them are
 # child dependencies of "gfx". Therefore, we separate it out here so multiple
 # targets can all have a dependency for header checking purposes without
 # creating circular dependencies.
 source_set("gfx_export") {
   sources = [
     "gfx_export.h",
   ]
 }
 
@@ skipping 524 matching lines @@
     ]
   }
 
   defines = [ "GFX_IMPLEMENTATION" ]
 
   deps = [
     ":native_widget_types",
     "//base",
     "//ui/gfx/geometry",
   ]
+
+  if (is_linux) {
+    sources += [
+      "linux/client_native_pixmap_dmabuf.cc",
+      "linux/client_native_pixmap_dmabuf.h",
+      "linux/client_native_pixmap_factory_dmabuf.cc",
+      "linux/client_native_pixmap_factory_dmabuf.h",
+    ]
+
+    if (use_sysroot || use_ozone) {
+      deps += [ "//third_party/libdrm" ]

[spang, 2017/03/15 15:40:32]

I think this conditional should be removed and just always depend on
//third_party/libdrm; there's already a conditional and pkg_config() there which
we shouldn't duplicate here.

[Julien Isorce, 2017/03/21 17:59:54]

Done, thx.

+    } else {
+      configs += [ ":libdrm" ]
+    }
+  }
 }
 
 static_library("test_support") {
   testonly = true
   sources = [
     "geometry/test/rect_test_util.cc",
     "geometry/test/rect_test_util.h",
     "image/image_unittest_util.cc",
     "image/image_unittest_util.h",
     "image/image_unittest_util_ios.mm",
@@ skipping 206 matching lines @@
 
 fuzzer_test("color_transform_fuzztest") {
   sources = [
     "color_transform_fuzzer.cc",
   ]
   deps = [
     ":gfx",
   ]
   libfuzzer_options = [ "max_len=1024" ]
 }