Revert "Add test directory -> monorail component metadata to test inv…

…ocations."

This reverts commit 39b25a5.

Reason for revert: broke Win7 Tests x64 bot (https://ci.chromium.org/p/chromium/builders/ci/Win%207%20Tests%20x64%20(1)/78401?)

Original change's description:
> Add test directory -> monorail component metadata to test invocations.
>
> The new ResultDB system has the ability to associate test results with
> Monorail components and teams, but in order to do that, we need to
> upload the mapping information contained in the testing/DIR_METADATA
> files along with the test results.
>
> This CL adds a gclient hook to generate the metadata in a
> ResultDB-friendly format that can be accessed by `rdb` and uploaded
> as part of a test invocation, and makes the generated data a
> dependency of the bin/run_* test scripts, to ensure that every
> test invocation will have the data.
>
> Change-Id: I83dcf7f460b6fbd1d4c23616d972510712019148
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2568896
> Commit-Queue: Dirk Pranke <dpranke@google.com>
> Reviewed-by: Chan Li <chanli@chromium.org>
> Reviewed-by: Nodir Turakulov <nodir@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#861612}

Change-Id: I024e516400daec4eb22edf2e0539baa1553948bc
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2749775
Auto-Submit: Dirk Pranke <dpranke@google.com>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#861642}

.gitignore

@@ -281,7 +281,6 @@ vs-chromium-project.txt
 /sql/sql_unittests_run.xml
 /sync/sync.xml
 /testing/libfuzzer/fuzzer_corpus_for_bots/
-/testing/location_tags.json
 /testing/rts/
 /testserver.log
 # See third_party/.gitignore for entries covering src/third_party.

DEPS

@@ -4366,17 +4366,6 @@ hooks = [
     ],
   },
 
-  {
-    'name': 'Generate component metadata for tests',
-    'pattern': '.',
-    'action': [
-      'vpython',
-      'src/testing/generate_location_tags.py',
-      '--out',
-      'src/testing/location_tags.json',
-    ],
-  },
-
   # Download and initialize "vpython" VirtualEnv environment packages.
   {
     'name': 'vpython_common',

build/config/fuchsia/generate_runner_scripts.gni

@@ -91,10 +91,6 @@ template("fuchsia_package_runner") {
       data = []
     }
 
-    if (defined(invoker.data)) {
-      data += invoker.data
-    }
-
     wrapper_script = generated_run_pkg_script_path
 
     data_deps = [

testing/test.gni

@@ -78,13 +78,11 @@ template("test") {
       executable(_exec_target) {
         # Configs will always be defined since we set_defaults in BUILDCONFIG.gn.
         configs = []
-        forward_variables_from(
-            invoker,
-            "*",
-            TESTONLY_AND_VISIBILITY + _wrapper_script_vars + [
-                  "data_deps",
-                  "extra_dist_files",
-                ])
+        data_deps = []
+        forward_variables_from(invoker,
+                               "*",
+                               TESTONLY_AND_VISIBILITY + _wrapper_script_vars +
+                                   [ "extra_dist_files" ])
 
         # Thanks to the set_defaults() for test(), configs are initialized with
         # the default shared_library configs rather than executable configs.
@@ -94,16 +92,6 @@ template("test") {
         ]
         configs += [ "//build/config:executable_config" ]
 
-        if (defined(invoker.data_deps)) {
-          data_deps = invoker.data_deps
-        } else {
-          data_deps = []
-        }
-        if (!defined(data)) {
-          data = []
-        }
-        data += [ "//testing/location_tags.json" ]
-
         # Don't output to the root or else conflict with the group() below.
         output_name = rebase_path(_exec_output, root_out_dir)
       }
@@ -237,13 +225,6 @@ template("test") {
           ":$_library_target",
         ]
       }
-
-      if (defined(invoker.data_deps)) {
-        data_deps = invoker.data_deps
-      } else {
-        data_deps = []
-      }
-      data = [ "//testing/location_tags.json" ]
     }
   } else if (is_fuchsia) {
     assert(!defined(invoker.use_xvfb) || !invoker.use_xvfb)
@@ -298,7 +279,6 @@ template("test") {
       package = ":$_pkg_target"
       package_name_override = _output_name
 
-      data = [ "//testing/location_tags.json" ]
       data_deps = [ "//testing/buildbot/filters:fuchsia_filters" ]
     }
 
@@ -336,6 +316,7 @@ template("test") {
       forward_variables_from(invoker,
                              [
                                "data",
+                               "data_deps",
                                "deps",
                                "executable_args",
                                "retries",
@@ -353,11 +334,6 @@ template("test") {
       ]
 
       wrapper_output_name = "${_wrapper_output_name}"
-
-      if (!defined(data)) {
-        data = []
-      }
-      data += [ "//testing/location_tags.json" ]
     }
 
     _resources_bundle_data = target_name + "_resources_bundle_data"
@@ -428,7 +404,6 @@ template("test") {
       generated_script = "$root_build_dir/bin/run_" + invoker.target_name
       test_exe = invoker.target_name
       runtime_deps_file = _runtime_deps_file
-      data = [ "//testing/location_tags.json" ]
     }
 
     executable(target_name) {
@@ -478,7 +453,6 @@ template("test") {
         executable = "//testing/test_env.py"
         data += [ "//testing/test_env.py" ]
       }
-      data += [ "//testing/location_tags.json" ]
 
       executable_args = [
         "@WrappedPath(../../build/lacros/test_runner.py)",
@@ -554,7 +528,6 @@ template("test") {
         executable = "//testing/test_env.py"
         data += [ "//testing/test_env.py" ]
       }
-      data += [ "//testing/location_tags.json" ]
 
       executable_args = [
         "@WrappedPath(./${_executable})",
@@ -658,8 +631,6 @@ template("script_test") {
     if (defined(invoker.data)) {
       data += invoker.data
     }
-    data += [ "//testing/location_tags.json" ]
-
     data_deps = []
     if (defined(invoker.data_deps)) {
       data_deps += invoker.data_deps