tools: use PrintCaughtException in the snapshot builder

This prints not only the error message but also the error source line and the stack trace wherever possible. PR-URL: #38745 Reviewed-By: Chengzhong Wu <[email protected]> Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Minwoo Jung <[email protected]>
nodejs · May 31, 2021 · ab44106 · ab44106
1 parent 1f5baaa
commit ab44106
Showing 1 changed file with 23 additions and 35 deletions.
diff --git a/tools/snapshot/snapshot_builder.cc b/tools/snapshot/snapshot_builder.cc
@@ -3,6 +3,7 @@
 #include <sstream>
 #include "debug_utils-inl.h"
 #include "env-inl.h"
+#include "node_errors.h"
 #include "node_external_reference.h"
 #include "node_internals.h"
 #include "node_main_instance.h"
@@ -15,10 +16,8 @@ using v8::Context;
 using v8::HandleScope;
 using v8::Isolate;
 using v8::Local;
-using v8::Object;
 using v8::SnapshotCreator;
 using v8::StartupData;
-using v8::String;
 using v8::TryCatch;
 using v8::Value;
 
@@ -112,55 +111,44 @@ std::string SnapshotBuilder::Generate(
       creator.SetDefaultContext(Context::New(isolate));
       isolate_data_indexes = main_instance->isolate_data()->Serialize(&creator);
 
-      TryCatch bootstrapCatch(isolate);
-      Local<Context> context = NewContext(isolate);
-      if (bootstrapCatch.HasCaught()) {
-        Local<Object> obj = bootstrapCatch.Exception()->ToObject(context)
-            .ToLocalChecked();
-        Local<Value> stack = obj->Get(
-            context,
-            FIXED_ONE_BYTE_STRING(isolate, "stack")).ToLocalChecked();
-        if (stack->IsUndefined()) {
-          Local<String> str = obj->Get(
-              context,
-              FIXED_ONE_BYTE_STRING(isolate, "name"))
-            .ToLocalChecked()->ToString(context).ToLocalChecked();
-          str = String::Concat(
-            isolate,
-            str,
-            FIXED_ONE_BYTE_STRING(isolate, ": "));
-          stack = String::Concat(
-            isolate,
-            str,
-            obj->Get(
-                context,
-                FIXED_ONE_BYTE_STRING(isolate, "message"))
-              .ToLocalChecked()->ToString(context).ToLocalChecked());
+      // Run the per-context scripts
+      Local<Context> context;
+      {
+        TryCatch bootstrapCatch(isolate);
+        context = NewContext(isolate);
+        if (bootstrapCatch.HasCaught()) {
+          PrintCaughtException(isolate, context, bootstrapCatch);
+          abort();
         }
-        v8::String::Utf8Value utf8_value(isolate, stack);
-        if (*utf8_value != nullptr) {
-          std::string out(*utf8_value, utf8_value.length());
-          fprintf(stderr, "Had Exception: %s\n", out.c_str());
-        } else {
-          fprintf(stderr, "Unknown JS Exception\n");
-        }
-        abort();
       }
       Context::Scope context_scope(context);
 
+      // Create the environment
       env = new Environment(main_instance->isolate_data(),
                             context,
                             args,
                             exec_args,
                             nullptr,
                             node::EnvironmentFlags::kDefaultFlags,
                             {});
-      env->RunBootstrapping().ToLocalChecked();
+      // Run scripts in lib/internal/bootstrap/
+      {
+        TryCatch bootstrapCatch(isolate);
+        v8::MaybeLocal<Value> result = env->RunBootstrapping();
+        if (bootstrapCatch.HasCaught()) {
+          PrintCaughtException(isolate, context, bootstrapCatch);
+        }
+        result.ToLocalChecked();
+      }
+
       if (per_process::enabled_debug_list.enabled(DebugCategory::MKSNAPSHOT)) {
         env->PrintAllBaseObjects();
         printf("Environment = %p\n", env);
       }
+
+      // Serialize the native states
       env_info = env->Serialize(&creator);
+      // Serialize the context
       size_t index = creator.AddContext(
           context, {SerializeNodeContextInternalFields, env});
       CHECK_EQ(index, NodeMainInstance::kNodeContextIndex);