ozone gbm: whitelist 3 DRM ioctl code for native GpuMemoryBuffer.

content/common/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/features.gni")
 import("//build/config/ui.gni")
 import("//content/common/common.gni")
 import("//third_party/mojo/src/mojo/public/tools/bindings/mojom.gni")
+import("//ui/ozone/ozone.gni")

[brettw, 2015/08/24 21:53:38]

A better way to do this would be to add a config
//ui/ozone:vgem_map

and defines USE_VGEM_MAP or not depending on the value of the flag. That way you
don't need to import this file and logic about these defines isn't duplicated.

[dshwang, 2015/08/26 14:18:02]

AFAIK, there is no way to inherit DEFINE value from dependencies.

[brettw, 2015/08/26 22:27:49]

In ui/ozone/BUILD.gn:
  config("vgem_map") {
    if (use_vgem_map) {
      defines = [ "USE_VGEM_MAP" ]
    }
  }

and in the targets you want to define this define in:
  configs += [ "//ui/ozone:vgem_map" ]
in all cases.

Now the conditional logic and define value is in one place.

[dshwang, 2015/08/27 12:47:09]

Oh, there is a way. Thank you for explaining!

Now define USE_VGEM_MAP only once in both gn and gyp

+
 if (is_mac) {
   import("//build/config/mac/mac_sdk.gni")
 }
 
 if (is_chromeos && current_cpu != "arm") {
   action("libva_generate_stubs") {
     extra_header = "gpu/media/va_stub_header.fragment"
 
     script = "../../tools/generate_stubs/generate_stubs.py"
     sources = [
@@ skipping 437 matching lines @@
 
     # TODO(GYP): extract_xinput action.
   }
 
   if (!is_win || !use_aura) {
     sources -= [ "cursors/webcursor_aurawin.cc" ]
   }
 
   if (use_seccomp_bpf) {
     defines += [ "USE_SECCOMP_BPF" ]
+
+    if (use_vgem_map) {
+      defines += [ "USE_VGEM_MAP" ]
+    }
   } else {
     if (is_linux) {
       sources -= [
         "sandbox_linux/bpf_cros_arm_gpu_policy_linux.cc",
         "sandbox_linux/bpf_cros_arm_gpu_policy_linux.h",
         "sandbox_linux/bpf_gpu_policy_linux.cc",
         "sandbox_linux/bpf_gpu_policy_linux.h",
         "sandbox_linux/bpf_ppapi_policy_linux.cc",
         "sandbox_linux/bpf_ppapi_policy_linux.h",
         "sandbox_linux/bpf_renderer_policy_linux.cc",
@@ skipping 30 matching lines @@
 
   import_dirs = [ "//mojo/services" ]
 
   deps = [
     "//content/public/common:mojo_bindings",
     "//mojo/application/public/interfaces",
     "//skia/public/interfaces",
     "//ui/mojo/geometry:interfaces",
   ]
 }