Run gl_unittests with Ozone X11.

ui/gl/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/buildflag_header.gni")
 import("//build/config/chrome_build.gni")
 import("//build/config/linux/pkg_config.gni")
 import("//build/config/ui.gni")
 import("//ui/ozone/ozone.gni")
 import("//testing/test.gni")
@@ skipping 386 matching lines @@
     "//ui/gfx",
     "//ui/gfx/geometry",
     "//ui/gl/init",
   ]
 
   data_deps = [
     "//third_party/mesa:osmesa",
   ]
 }
 
+group("gl_unittests_ozonex") {
+  testonly = true
+  data_deps = [
+    ":gl_unittests",
+  ]
+}
+
+# TODO(GYP_GONE): Delete this after we've converted everything to GN.

[sadrul, 2016/11/14 21:19:35]

GYP_IS_ALREADY_GONE I hope?

[kylechar, 2016/11/14 21:49:00]

Yes, GYP is gone but GN still needs the extra target. Not totally sure why it
does, but all of the <test>_run targets still have this comment. Like 
https://cs.chromium.org/chromium/src/chrome/test/BUILD.gn?q=%22TODO(GYP_GONE)...
for example.

This way gl_unittests_ozonex_run gets cleaned up with the rest of them, should
https://cs.chromium.org/chromium/src/testing/test.gni?q=%22TODO(GYP_GONE)%22&...
change and not require <test>_run anymore?

+# The _run targets exist only for compatibility w/ GYP.
+group("gl_unittests_ozonex_run") {
+  testonly = true
+  deps = [
+    ":gl_unittests_ozonex",
+  ]
+}
+
 if (is_android) {
   generate_jar_jni("surface_jni_headers") {
     jni_package = "ui/gl"
     classes = [ "android/view/Surface.class" ]
   }
 
   generate_jni("gl_jni_headers") {
     sources = [
       "../android/java/src/org/chromium/ui/gl/SurfaceTextureListener.java",
       "../android/java/src/org/chromium/ui/gl/SurfaceTexturePlatformWrapper.java",
     ]
     public_deps = [
       ":surface_jni_headers",
     ]
     jni_package = "ui/gl"
   }
 }