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

Unified Diff: test/mjsunit/es6/promises.js

Issue 1536013002: [promise] make Promise.resolve match spec (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Update test expectations Created 5 years 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: test/mjsunit/es6/promises.js
diff --git a/test/mjsunit/es6/promises.js b/test/mjsunit/es6/promises.js
index d02f7b7ef3e869e3ffede7bba2a9f3308044a601..ccc5649a9c51fd36a1ab1b8566acfaaefdee719c 100644
--- a/test/mjsunit/es6/promises.js
+++ b/test/mjsunit/es6/promises.js
@@ -47,6 +47,8 @@ function clear(o) {
clear(o.__proto__)
var properties = getOwnPropertyNames(o)
for (var i in properties) {
+ // Do not clobber Object.prototype.toString, which is used by tests.
+ if (properties[i] === "toString") continue;
clearProp(o, properties[i])
}
}
@@ -1055,7 +1057,8 @@ function assertAsyncDone(iteration) {
log = ""
MyPromise.all([21, Promise.accept(22), 23, MyPromise.accept(24), 25, 26])
- assertTrue(log === "nx24nnx21nnnnx23nnnx25nnx26n", "subclass/all/self")
+ assertTrue(log === "nx24nnx21nnx[object Promise]nnx23nnnx25nnx26n",
caitp (gmail) 2015/12/22 17:34:06 This result matches the current behaviour of JavaS
+ "subclass/all/self")
})();
(function() {
« src/js/promise.js ('K') | « src/js/promise.js ('k') | test/test262/test262.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698