Move the kernel isolate snapshot out of the 'runtime' target to reduce build times on the simulator…

BUILD.gn

 # Copyright (c) 2016, the Dart project authors.  Please see the AUTHORS file
 # for details. All rights reserved. Use of this source code is governed by a
 # BSD-style license that can be found in the LICENSE file.
 
 # This target will be built if no target is specified when invoking ninja.
 group("default") {
   if (defined(is_fuchsia) && (is_fuchsia || is_fuchsia_host)) {
     # Fuchsia has run_vm_tests marked testonly.
     testonly = true
   }
@@ skipping 24 matching lines @@
     testonly = true
   }
   deps = [
     "runtime/bin:dart",
     "runtime/bin:dart_bootstrap($host_toolchain)",
     "runtime/bin:process_test",
     "runtime/bin:run_vm_tests",
     "runtime/bin:sample_extension",
     "runtime/bin:test_extension",
     "runtime/vm:patched_sdk",
+  ]
+}
+
+group("runtime_kernel") {
+  deps = [
+    ":runtime",

[jamesr1, 2017/01/31 23:10:02]

unfortunately this breaks the fuchsia GN build as this non-testonly target now
depends on a target marked testonly in fuchsia, see line 35.  Why does
//dart:runtime depend on run_vm_tests?  Is that not actually a test-only target
or should //dart:runtime (and thus //dart:runtime_kernel) be considered testonly
as well?

[rmacnak, 2017/01/31 23:26:10]

runtime and runtime_kernel should have the same testonly-ness.

I believe the grouping of run_vm_tests under runtime is mostly historical: we
did it that way in GYP, our bots build the "runtime" target and we're keeping GN
and GYP roughly in sync until we can stop supporting GN. This probably should be
refactored after that.

 
     # TODO(rmacnak): Link this into 'dart'
     "utils/kernel-service:kernel-service",
   ]
 }
 
 group("runtime_precompiled") {
   deps = [
     "runtime/bin:dart_bootstrap($host_toolchain)",
     "runtime/bin:dart_precompiled_runtime",
@@ skipping 184 matching lines @@
         rebase_path(input),
         "--output-file",
         rebase_path(output),
         "--build-id-map",
         rebase_path("$target_gen_dir/build_id_map"),
         "--compress",
       ]
     }
   }
 }