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

Unified Diff: src/js/promise.js

Issue 2450763002: [promises] remove PromiseEnqueue (Closed)
Patch Set: Created 4 years, 2 months 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/js/promise.js
diff --git a/src/js/promise.js b/src/js/promise.js
index af1336c0141a9364bf07d13a321874243ee0d46e..b5437aefe7495db6434dbd38927be0b118395bf6 100644
--- a/src/js/promise.js
+++ b/src/js/promise.js
@@ -147,7 +147,7 @@ function FulfillPromise(promise, status, value, promiseQueue) {
var tasks = GET_PRIVATE(promise, promiseQueue);
if (!IS_UNDEFINED(tasks)) {
var deferred = GET_PRIVATE(promise, promiseDeferredReactionSymbol);
- PromiseEnqueue(value, tasks, deferred, status);
+ %EnqueuePromiseReactionJob(value, tasks, deferred, status);
}
PromiseSet(promise, status, value);
}
@@ -178,7 +178,7 @@ function PromiseHandle(value, handler, deferred) {
}
}
-function PromiseEnqueue(value, tasks, deferreds, status) {
+function PromiseDebugGetInfo(deferreds, status) {
var id, name, instrumenting = DEBUG_IS_ACTIVE;
if (instrumenting) {
@@ -201,7 +201,7 @@ function PromiseEnqueue(value, tasks, deferreds, status) {
%DebugAsyncTaskEvent("enqueue", id, name);
}
}
- %EnqueuePromiseReactionJob(value, tasks, deferreds, id, name);
+ return [id, name];
}
function PromiseAttachCallbacks(promise, deferred, onResolve, onReject) {
@@ -390,7 +390,7 @@ function PerformPromiseThen(promise, onResolve, onReject, resultCapability) {
PromiseAttachCallbacks(promise, resultCapability, onResolve, onReject);
break;
case kFulfilled:
- PromiseEnqueue(GET_PRIVATE(promise, promiseResultSymbol),
+ %EnqueuePromiseReactionJob(GET_PRIVATE(promise, promiseResultSymbol),
onResolve, resultCapability, kFulfilled);
break;
case kRejected:
@@ -399,7 +399,7 @@ function PerformPromiseThen(promise, onResolve, onReject, resultCapability) {
// Revoke previously triggered reject event.
%PromiseRevokeReject(promise);
}
- PromiseEnqueue(GET_PRIVATE(promise, promiseResultSymbol),
+ %EnqueuePromiseReactionJob(GET_PRIVATE(promise, promiseResultSymbol),
onReject, resultCapability, kRejected);
break;
}
@@ -659,7 +659,8 @@ utils.InstallFunctions(GlobalPromise.prototype, DONT_ENUM, [
"promise_reject", DoRejectPromise,
"promise_resolve", ResolvePromise,
"promise_then", PromiseThen,
- "promise_handle", PromiseHandle
+ "promise_handle", PromiseHandle,
+ "promise_debug_get_info", PromiseDebugGetInfo
]);
// This allows extras to create promises quickly without building extra
« no previous file with comments | « src/contexts.h ('k') | src/runtime/runtime.h » ('j') | src/runtime/runtime-internal.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698