Merged: Avoid calling out to JS during stack overflow

src/isolate.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/isolate.h"
 
 #include <stdlib.h>
 
 #include <fstream>  // NOLINT(readability/streams)
 #include <sstream>
@@ skipping 1717 matching lines @@
   PromiseOnStack* prev = tltop->promise_on_stack_->prev();
   Handle<Object> global_promise = tltop->promise_on_stack_->promise();
   delete tltop->promise_on_stack_;
   tltop->promise_on_stack_ = prev;
   global_handles()->Destroy(global_promise.location());
 }
 
 bool Isolate::PromiseHasUserDefinedRejectHandler(Handle<Object> promise) {
   Handle<JSFunction> fun = promise_has_user_defined_reject_handler();
   Handle<Object> has_reject_handler;
+  // If we are, e.g., overflowing the stack, don't try to call out to JS
+  if (!AllowJavascriptExecution::IsAllowed(this)) return false;
+  // Call the registered function to check for a handler
   if (Execution::TryCall(this, fun, promise, 0, NULL)
           .ToHandle(&has_reject_handler)) {
     return has_reject_handler->IsTrue(this);
   }
   // If an exception is thrown in the course of execution of this built-in
   // function, it indicates either a bug, or a synthetic uncatchable
   // exception in the shutdown path. In either case, it's OK to predict either
   // way in DevTools.
   return false;
 }
@@ skipping 1569 matching lines @@
   // Then check whether this scope intercepts.
   if ((flag & intercept_mask_)) {
     intercepted_flags_ |= flag;
     return true;
   }
   return false;
 }
 
 }  // namespace internal
 }  // namespace v8