[tools/perf] Make telemetry_chrome_test depend on trace_processor_shell

Some tests depend on "telemetry_chrome_test" target directly, bypassing
"perf" target. This CL moves the dependency on trace_processor_shell from
"perf" to "telemetry_chrome_test" to make sure that all tests can use the
trace processor to convert proto traces.

Bug: 1038780
Change-Id: I308024cd5ec06bd4989ff983e28e9f26b19f792c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1983173Reviewed-by: Caleb Rouleau <crouleau@chromium.org>
Commit-Queue: Mikhail Khokhlov <khokhlov@google.com>
Cr-Commit-Position: refs/heads/master@{#728520}

tools/perf/BUILD.gn

@@ -8,10 +8,6 @@ group("perf") {
   deps = [
     "//tools/perf/chrome_telemetry_build:telemetry_chrome_test",
   ]
-  data_deps = [
-    "//third_party/perfetto/src/trace_processor:trace_processor_shell",
-  ]
-
   data = [
     "//tools/perf/",
 
@@ -39,7 +35,9 @@ group("perf") {
 
   # Runs a script which generates the ad tagging ruleset.
   if (!is_ios) {
-    data_deps += [ "//components/subresource_filter/tools:index_ruleset" ]
+    data_deps = [
+      "//components/subresource_filter/tools:index_ruleset",
+    ]
   }
 }
 

tools/perf/chrome_telemetry_build/BUILD.gn

@@ -16,6 +16,7 @@ group("telemetry_chrome_test") {
 
   data_deps = [
     ":telemetry_chrome_test_without_chrome",
+    "//third_party/perfetto/src/trace_processor:trace_processor_shell",
   ]
   data = []