report: disable js stack when no context is entered

There are no guarantees that the JS stack can be generated when no
context is entered.

Author: legendecas

src/node_report.cc

@@ -452,8 +452,9 @@ static void PrintEmptyJavaScriptStack(JSONWriter* writer) {
 static void PrintJavaScriptStack(JSONWriter* writer,
                                  Isolate* isolate,
                                  const char* trigger) {
-  // Can not capture the stacktrace when the isolate is in a OOM state.
-  if (!strcmp(trigger, "OOMError")) {
+  // Can not capture the stacktrace when the isolate is in a OOM state or no
+  // context is entered.
+  if (!strcmp(trigger, "OOMError") || !isolate->InContext()) {
     PrintEmptyJavaScriptStack(writer);
     return;
   }

test/addons/report-api/test.js

@@ -9,7 +9,7 @@ const tmpdir = require('../../common/tmpdir');
 const binding = path.resolve(__dirname, `./build/${common.buildType}/binding`);
 const addon = require(binding);
 
-function myAddonMain(method, { hasIsolate, hasEnv }) {
+function myAddonMain(method, { hasContext, hasEnv }) {
   tmpdir.refresh();
   process.report.directory = tmpdir.path;
 
@@ -27,7 +27,7 @@ function myAddonMain(method, { hasIsolate, hasEnv }) {
   const content = require(report);
 
   // Check that the javascript stack is present.
-  if (hasIsolate) {
+  if (hasContext) {
     assert.strictEqual(content.javascriptStack.stack.findIndex((frame) => frame.match('myAddonMain')), 0);
   } else {
     assert.strictEqual(content.javascriptStack, undefined);
@@ -45,9 +45,9 @@ const methods = [
   ['triggerReportNoIsolate', false, false],
   ['triggerReportEnv', true, true],
   ['triggerReportNoEnv', false, false],
-  ['triggerReportNoContext', true, false],
+  ['triggerReportNoContext', false, false],
   ['triggerReportNewContext', true, false],
 ];
-for (const [method, hasIsolate, hasEnv] of methods) {
-  myAddonMain(method, { hasIsolate, hasEnv });
+for (const [method, hasContext, hasEnv] of methods) {
+  myAddonMain(method, { hasContext, hasEnv });
 }