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

Unified Diff: src/promise.js

Issue 416213004: Merge three PromiseEvent's into one. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Fix formatting Created 6 years, 5 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/debug-debugger.js ('k') | test/mjsunit/es6/debug-promise-events.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/promise.js
diff --git a/src/promise.js b/src/promise.js
index 09582a279758b1ba43fb4991c567cc784ca5a211..0c930e6338d85746fe2b8ea187b6439b798da16c 100644
--- a/src/promise.js
+++ b/src/promise.js
@@ -39,11 +39,6 @@ var lastMicrotaskId = 0;
if (!IS_SPEC_FUNCTION(resolver))
throw MakeTypeError('resolver_not_a_function', [resolver]);
var promise = PromiseInit(this);
- if (DEBUG_IS_ACTIVE) {
- %DebugPromiseEvent({ type : "new",
- promise: this,
- resolver: resolver });
- }
try {
%DebugPromiseHandlePrologue(function() { return promise });
resolver(function(x) { PromiseResolve(promise, x) },
@@ -62,11 +57,8 @@ var lastMicrotaskId = 0;
SET_PRIVATE(promise, promiseValue, value);
SET_PRIVATE(promise, promiseOnResolve, onResolve);
SET_PRIVATE(promise, promiseOnReject, onReject);
- if (DEBUG_IS_ACTIVE && status !== 0) {
- %DebugPromiseEvent({ type: "update",
- promise: promise,
- status: status,
- value: value });
+ if (DEBUG_IS_ACTIVE) {
+ %DebugPromiseEvent({ promise: promise, status: status, value: value });
}
return promise;
}
@@ -241,9 +233,7 @@ var lastMicrotaskId = 0;
break;
}
if (DEBUG_IS_ACTIVE) {
- %DebugPromiseEvent({ type: "chain",
- promise: deferred.promise,
- parentPromise: this });
+ %DebugPromiseEvent({ promise: deferred.promise, parentPromise: this });
}
return deferred.promise;
}
« no previous file with comments | « src/debug-debugger.js ('k') | test/mjsunit/es6/debug-promise-events.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698