Disable node snapshots (#245)

A security fix in V8 makes it impossible to put FunctionTemplateInfo
objects of API functions with fast API callbacks into the snapshot. As
Node.js does that so far, this PR disables node snapshots. In the
future, FunctionTemplateInfo objects without fast API callbacks could be
added back to the snapshot.

Author: gahaas

node.gni

@@ -68,7 +68,12 @@ declare_args() {
   # TODO(zcbenz): There are few broken things for now:
   #   1. cross-os compilation is not supported.
   #   2. node_mksnapshot crashes when cross-compiling for x64 from arm64.
-  node_use_node_snapshot = (host_os == target_os) && !(host_cpu == "arm64" && target_cpu == "x64")
+  # node_use_node_snapshot = (host_os == target_os) && !(host_cpu == "arm64" && target_cpu == "x64")
+
+  # Disable snapshot for now. Node.js currently puts `FunctionTemplateInfo`
+  # objects into the snapshot, but that only works for `FunctionTemplateInfo`
+  # objects of API functions that do not support fast API calls.
+  node_use_node_snapshot = false
 
   # Build with Amaro (TypeScript utils).
   node_use_amaro = true