Rename wrapper_scripts for generated_script tests in MB.

A few tests (some Android resource_sizes tests, and one Perfetto
test), were using generated_script tests that were generating
wrappers that weren't named 'bin/run_$target_name'.

We want all generated_script tests to be using consistent names, so
this CL changes those targets.

Bug: 816629
Change-Id: If447fbee6b33637de4c9711e87379128272285af
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380672
Commit-Queue: Dirk Pranke <dpranke@google.com>
Reviewed-by: John Chen <johnchen@chromium.org>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Cr-Commit-Position: refs/heads/master@{#802694}

build/android/resource_sizes.gni

@@ -19,7 +19,7 @@ template("android_resource_sizes_test") {
   generate_android_wrapper(target_name) {
     forward_variables_from(invoker, [ "data_deps" ])
     executable = "//build/android/resource_sizes.py"
-    wrapper_script = "$root_out_dir/bin/${target_name}"
+    wrapper_script = "$root_out_dir/bin/run_${target_name}"
 
     assert(defined(invoker.apk_name) != defined(invoker.file_path),
            "Exactly one of apk_name or file_path should be provided.")

testing/buildbot/gn_isolate_map.pyl

@@ -46,11 +46,9 @@
 #  : the test is a python script; the path to the script is specified in
 #    the "script" field.
 #  "generated_script"
-#  : the test is a script generated at build time; the script *should* be
+#  : the test is a script generated at build time; the script *must* be
 #    in output_dir/bin/run_$target (or output_dir\bin\run_$target.bat on
-#    Windows), but an alternative path *may* be specified,
-#    in the "script" field, as *relative to the output directory*.
-#    TODO(crbug.com/816629): remove support for "script".
+#    Windows).
 #  "unknown"
 #  : (the default), which indicates that we don't know what the command line
 #    needs to be (this is a fatal error).
@@ -1423,27 +1421,22 @@
   "resource_sizes_chrome_modern_public_minimal_apks": {
     "label": "//chrome/android:resource_sizes_chrome_modern_public_minimal_apks",
     "type": "generated_script",
-    "script": "bin/resource_sizes_chrome_modern_public_minimal_apks",
   },
   "resource_sizes_cronet_sample_apk": {
     "label": "//components/cronet/android:resource_sizes_cronet_sample_apk",
     "type": "generated_script",
-    "script": "bin/resource_sizes_cronet_sample_apk",
   },
   "resource_sizes_monochrome_public_minimal_apks": {
     "label": "//chrome/android:resource_sizes_monochrome_public_minimal_apks",
     "type": "generated_script",
-    "script": "bin/resource_sizes_monochrome_public_minimal_apks",
   },
   "resource_sizes_trichrome": {
     "label": "//chrome/android:resource_sizes_trichrome",
     "type": "generated_script",
-    "script": "bin/resource_sizes_trichrome",
   },
   "resource_sizes_system_webview_bundle": {
     "label": "//android_webview:resource_sizes_system_webview_bundle",
     "type": "generated_script",
-    "script": "bin/resource_sizes_system_webview_bundle",
   },
   "sandbox_linux_unittests": {
     "label": "//sandbox/linux:sandbox_linux_unittests",
@@ -1676,7 +1669,6 @@
   "upload_trace_processor": {
     "label": "//tools/perf/core/perfetto_binary_roller:upload_trace_processor",
     "type": "generated_script",
-    "script": "bin/upload_trace_processor",
   },
   "url_unittests": {
     "label": "//url:url_unittests",

tools/mb/mb.py

@@ -1388,6 +1388,8 @@ class MetaBuildWrapper(object):
       cmdline = []
 
     if test_type == 'generated_script' or is_ios or is_lacros:
+      assert 'script' not in isolate_map[target], (
+          'generated_scripts can no longer customize the script path')
       if is_win:
         default_script = 'bin\\run_{}.bat'.format(target)
       else:

tools/perf/core/perfetto_binary_roller/BUILD.gn

@@ -22,7 +22,7 @@ generate_wrapper("upload_trace_processor") {
   }
 
   executable = "upload_trace_processor.py"
-  wrapper_script = "$root_build_dir/bin/upload_trace_processor"
+  wrapper_script = "$root_build_dir/bin/run_upload_trace_processor"
   executable_args = [
     "--path",
     "@WrappedPath(./$binary_name)",