fixup! more resilient?

Author: saghul

quickjs.c

@@ -1260,6 +1260,8 @@ static JSValue js_promise_resolve(JSContext *ctx, JSValueConst this_val,
                                   int argc, JSValueConst *argv, int magic);
 static JSValue js_promise_then(JSContext *ctx, JSValueConst this_val,
                                int argc, JSValueConst *argv);
+static JSValue js_promise_resolve_thenable_job(JSContext *ctx,
+                                               int argc, JSValueConst *argv);
 static bool js_string_eq(JSString *p1, JSString *p2);
 static int js_string_compare(JSString *p1, JSString *p2);
 static int JS_SetPropertyValue(JSContext *ctx, JSValueConst this_obj,
@@ -50223,7 +50225,7 @@ static JSValue promise_rejection_tracker_job(JSContext *ctx, int argc,
     JSValueConst promise;
     struct list_head *el, *el1;
     JSJobEntry *job;
-    bool has_other_jobs;
+    bool has_other_promise_jobs;
 
     assert(argc == 1);
 
@@ -50239,15 +50241,15 @@ static JSValue promise_rejection_tracker_job(JSContext *ctx, int argc,
     // Push the rejection tracker jobs to the end of the queue if there are other jobs.
     // This allows us to handle rejections that get added later and thus would handle the
     // rejection _after_ we check for it.
-    has_other_jobs = false;
+    has_other_promise_jobs = false;
     list_for_each_safe(el, el1, &rt->job_list) {
         job = list_entry(el, JSJobEntry, link);
-        if (job->job_func != promise_rejection_tracker_job) {
-            has_other_jobs = true;
+        if (job->job_func == promise_reaction_job || job->job_func == js_promise_resolve_thenable_job) {
+            has_other_promise_jobs = true;
             break;
         }
     }
-    if (has_other_jobs) {
+    if (has_other_promise_jobs) {
         JS_EnqueueJob(ctx, promise_rejection_tracker_job, 1, &promise);
         return JS_UNDEFINED;
     }