content_shell: Remove deps for resources only needed at runtime.

components/test_runner/BUILD.gn

Index: components/test_runner/BUILD.gn
diff --git a/components/test_runner/BUILD.gn b/components/test_runner/BUILD.gn
index 96827b19137c357e7d427a966354bb096e499df8..7938f70e63299adc642d87c9d9fc46d27059ef3c 100644
--- a/components/test_runner/BUILD.gn
+++ b/components/test_runner/BUILD.gn
@@ -107,7 +107,6 @@ component("test_runner") {
   configs += [ "//build/config/compiler:no_size_t_to_int_warning" ]
 
   deps = [
-    ":resources",
     "//base:base",
     "//base:i18n",
     "//cc",
@@ -190,23 +189,18 @@ if (is_mac) {
 }
 
 group("resources") {
-  deps = []
   data_deps = []
 
   if (is_mac) {
-    deps += [ ":test_runner_bundle_data" ]

[Robert Sesek, 2016/11/21 16:59:31]

I think this broke Mac content_shell. Did you try a clobber build and then run
WebKit layout tests?

[mithro, 2016/11/22 06:02:35]

No I did not. This seems to be passing fine on the bots?

[Robert Sesek, 2016/11/22 16:50:23]

The bots don't clobber by default. But your change here stops the AHEM fonts
from being copied into the Content Shell.app bundle, and after a clobber, this
will cause WebKit layout tests to fail.

I uploaded this dummy CL and send a `git cl try -c` (for clobber) to
demonstrate: https://codereview.chromium.org/2515253005/.

The test all hit this DLOG(FATAL):
https://cs.chromium.org/chromium/src/content/shell/app/blink_test_platform_su...

     data_deps += [ ":test_runner_bundle_data" ]
   } else {
-    deps += [ ":copy_ahem" ]
     data_deps += [ ":copy_ahem" ]
   }
 
   if (use_x11) {
-    deps += [ ":copy_x11_fonts" ]
     data_deps += [ ":copy_x11_fonts" ]
   }
   if (is_android) {
-    deps += [ ":copy_android_fonts" ]
     data_deps += [ ":copy_android_fonts" ]
   }
 }