Add goldctl dependency to browser_tests

This is needed for pixel tests which use goldctl.

Bug: 958242
Change-Id: Iebb879f3d98ec7d7044e135a89248363c6e65ad1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1788806
Auto-Submit: Sven Zheng <svenzheng@chromium.org>
Commit-Queue: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#694288}

chrome/test/BUILD.gn

@@ -1942,6 +1942,16 @@ if (!is_android) {
           [ "//chrome/browser/internal/resources/signin/test_accounts.json" ]
     }
 
+    # For pixel tests. Because this links to a CIPD
+    # dependency, which is a symlink on Unix platforms, refer to the
+    # actual executable rather than the whole directory; copying the
+    # whole directory doesn't work, at least with "mb.py zip".
+    if (is_win) {
+      data += [ "//tools/skia_goldctl/goldctl.exe" ]
+    } else {
+      data += [ "//tools/skia_goldctl/goldctl" ]
+    }
+
     if (is_chromeos) {
       assert(enable_app_list)
       sources += [