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

Unified Diff: LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html

Issue 24980002: Implement AP2 Promises (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Created 7 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: LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html
diff --git a/LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html b/LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html
index 8e4a002b4daf8ca02d1bb62f5047bb5969cb8d67..35a027e3711739ffb5b785c7b26b272286f3c029 100644
--- a/LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html
+++ b/LayoutTests/fast/js/Promise-resolve-with-then-fulfill.html
@@ -10,16 +10,14 @@
description('Test Promise.');
window.jsTestIsAsync = true;
-var callback;
var value = {
then: function(fulfillCallback, rejectCallback) {
testPassed('value.then is called.');
window.thisValue = this;
shouldBe('thisValue', 'value');
- callback = fulfillCallback;
+ fulfillCallback('hello');
}
};
-finishJSTest();
var promise = new Promise(function(resolve) { resolve(value); });
promise.then(function(result) {
@@ -34,7 +32,6 @@ promise.then(function(result) {
debug('The promise is not fulfilled now.');
-callback('hello');
</script>
<script src="resources/js-test-post.js"></script>
</body>

Powered by Google App Engine
This is Rietveld 408576698