Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(423)

Unified Diff: src/isolate.cc

Issue 2497523002: [promises] Move promise constructor to TFS (Closed)
Patch Set: fix test Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/isolate.cc
diff --git a/src/isolate.cc b/src/isolate.cc
index dadee9af50fd97269451ee8ce3a4b55ca3348df5..4c15568748e03c3587ecb80b8ac535db7169b13b 100644
--- a/src/isolate.cc
+++ b/src/isolate.cc
@@ -1341,6 +1341,10 @@ HandlerTable::CatchPrediction PredictException(JavaScriptFrame* frame) {
frame->Summarize(&summaries);
for (const FrameSummary& summary : summaries) {
Handle<AbstractCode> code = summary.abstract_code();
+ if (code->IsCode() && code->kind() == AbstractCode::BUILTIN &&
+ code->GetCode()->is_promise_rejection()) {
+ return HandlerTable::PROMISE;
+ }
if (code->kind() == AbstractCode::OPTIMIZED_FUNCTION) {
DCHECK(summary.function()->shared()->asm_function());
DCHECK(!FLAG_turbo_asm_deoptimization);
« src/builtins/builtins-promise.cc ('K') | « src/contexts.h ('k') | src/js/promise.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698