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

Unified Diff: src/js/promise.js

Issue 2353303003: [promises] Don't create resolving functions for PromiseCreate (Closed)
Patch Set: remove unnecessary function Created 4 years, 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/js/promise.js
diff --git a/src/js/promise.js b/src/js/promise.js
index 24ca74b531c36d5af8313e0b9ea510ead55d658b..348d5fb93ebf5de4625f7815629c0bca7ecef209 100644
--- a/src/js/promise.js
+++ b/src/js/promise.js
@@ -248,8 +248,6 @@ function PromiseIdResolveHandler(x) { return x; }
function PromiseIdRejectHandler(r) { %_ReThrow(r); }
SET_PRIVATE(PromiseIdRejectHandler, promiseForwardingHandlerSymbol, true);
-function PromiseNopResolver() {}
-
// -------------------------------------------------------------------
// Define exported functions.
@@ -261,7 +259,7 @@ function IsPromise(x) {
}
function PromiseCreate() {
- return new GlobalPromise(PromiseNopResolver);
+ return PromiseInit(new GlobalPromise(promiseRawSymbol));
}
// ES#sec-promise-resolve-functions
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698