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

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: src/isolate.cc

Issue 2497523002: [promises] Move promise constructor to TFS (Closed)
Patch Set: fixees 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
« src/builtins/x64/builtins-x64.cc ('K') | « src/factory.cc ('k') | src/js/promise.js » ('j') | src/objects.h » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/isolate.cc
diff --git a/src/isolate.cc b/src/isolate.cc
index dadee9af50fd97269451ee8ce3a4b55ca3348df5..7093a21f3099f096544f89c390232a5f310e116e 100644
--- a/src/isolate.cc
+++ b/src/isolate.cc
@@ -1341,6 +1341,9 @@ HandlerTable::CatchPrediction PredictException(JavaScriptFrame* frame) {
frame->Summarize(&summaries);
for (const FrameSummary& summary : summaries) {
Handle<AbstractCode> code = summary.abstract_code();
+ if (code->IsCode() && code->GetCode()->exception_hint() == 1) {
+ return HandlerTable::PROMISE;
+ }
if (code->kind() == AbstractCode::OPTIMIZED_FUNCTION) {
DCHECK(summary.function()->shared()->asm_function());
DCHECK(!FLAG_turbo_asm_deoptimization);
« src/builtins/x64/builtins-x64.cc ('K') | « src/factory.cc ('k') | src/js/promise.js » ('j') | src/objects.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698