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

Unified Diff: test/mjsunit/regress/regress-5476.js

Issue 2399053003: [promises] fix deferred object leak (Closed)
Patch Set: test reject case too 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
« no previous file with comments | « src/js/promise.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/regress/regress-5476.js
diff --git a/test/mjsunit/regress/regress-5476.js b/test/mjsunit/regress/regress-5476.js
new file mode 100644
index 0000000000000000000000000000000000000000..5d18cebf7a4700b862e4548a9177ceffe45c8f11
--- /dev/null
+++ b/test/mjsunit/regress/regress-5476.js
@@ -0,0 +1,16 @@
+// Copyright 2016 the V8 project authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+'use strict'
+
+class LeakyPromise extends Promise {
+ constructor(executor) {
+ super((resolve, reject) => { resolve();});
+ this.resolve = function() {assertEquals(this, undefined); };
+ this.reject = function() {assertEquals(this, undefined); };
+ executor(this.resolve, this.reject);
+ }
+}
+
+const p1 = new LeakyPromise((r) => r());
+const p2 = new LeakyPromise((_, r) => r());
« no previous file with comments | « src/js/promise.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698