Move gn _run target generation into test(). (RELAND)

sql/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("//testing/test.gni")
 
 component("sql") {
   sources = [
     "connection.cc",
     "connection.h",
@@ skipping 51 matching lines @@
   # This target exists because we need a way to switch between
   # "test/sql_test_base.h" and "mojo/sql_test_base.h" at compile time, to allow
   # us to switch out the gtest vs mojo:apptest frameworks.
   check_includes = false
 
   sources = [
     "correct_sql_test_base.h",
   ]
 }
 
-# TODO(GYP): Delete this after we've converted everything to GN.
-# The _run targets exist only for compatibility w/ GYP.
-group("sql_unittests_run") {
-  testonly = true
-  deps = [
-    ":sql_unittests",
-  ]
-}
-
 test("sql_unittests") {
   sources = [
     "connection_unittest.cc",
     "meta_table_unittest.cc",
     "recovery_unittest.cc",
     "sql_memory_dump_provider_unittest.cc",
     "sqlite_features_unittest.cc",
     "statement_unittest.cc",
     "test/paths.cc",
     "test/paths.h",
@@ skipping 38 matching lines @@
   #'target_name': 'sql_unittests_apk',
   #'type': 'none',
   #'dependencies': [
   #  'sql_unittests',
   #],
   #'variables': {
   #  'test_suite_name': 'sql_unittests',
   #},
   #'includes': [ '../build/apk_test.gypi' ],
 }