Move the test template to //testing/test.gni (part 2)

ui/base/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/ui.gni")
+import("//testing/rules.gni")
 
 if (is_android) {
   import("//build/config/android/config.gni")
   import("//build/config/android/rules.gni")
 }
 
 build_ime = !is_ios
 
 component("base") {
   output_name = "ui_base"
@@ skipping 827 matching lines @@
 
     # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
     cflags = [ "/wd4267" ]
   }
 
   if (!is_win || !use_aura) {
     sources -= [ "view_prop_unittest.cc" ]
   }
 
   if (is_android) {
-    deps += [
-      #"testing/android/native_test.gyp:native_test_native_code"  TODO(GYP)

[tfarina, 2015/01/12 18:44:59]

Why this is not necessary anymore? I was going to upload a CL fixing this TODO.

[qsr, 2015/01/12 18:48:54]

Because this CL change the test template so that we never need anything like
this anymore. As this is WiP, please feel free to fix that as you need, and I
will just remove it when this lands.

-    ]
+    apk_deps = [ "//chrome:resources" ]
   }
 
   if (use_pango) {
     configs += [ "//build/config/linux:pangocairo" ]
   }
 
   if (use_x11) {
     sources += [ "cursor/cursor_loader_x11_unittest.cc" ]
 
     configs += [ "//build/config/linux:x11" ]
@@ skipping 28 matching lines @@
     sources += [
       "dragdrop/os_exchange_data_provider_aurax11_unittest.cc",
       "x/selection_requestor_unittest.cc",
     ]
   }
 
   if (is_chromeos) {
     deps += [ "//chromeos" ]
   }
 }
-
-# TODO(GYP) Mac (ui_base_tests_bundle)

[tfarina, 2015/01/12 19:10:37]

btw, do not remove this Mac TODO. We still need to do it.

[qsr, 2015/01/12 19:15:52]

Will do. The comment was really weird, as it seemed to apply to an is_android
section, which made little sense.

-if (is_android) {
-  unittest_apk("ui_base_unittests_apk") {

[tfarina, 2015/01/12 18:44:59]

Why is is going away now?

[qsr, 2015/01/12 18:48:54]

Because this CL change the test template to automatically create the apk.

-    unittests_dep = ":ui_base_unittests"
-    deps = [
-      ":ui_base_unittests",
-      "//chrome:resources",
-    ]
-  }
-}