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

Unified Diff: test/mjsunit/es6/debug-promise-events.js

Issue 429453005: Do not include native Javascript in ExecutionState frames. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: rebase Created 6 years, 4 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/runtime.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/es6/debug-promise-events.js
diff --git a/test/mjsunit/es6/debug-promise-events.js b/test/mjsunit/es6/debug-promise-events.js
index 7e10835cfc40c9db24b648aa4984612d9056c48f..a9f94543f40594450e5ec02d9317a0e605b48f23 100644
--- a/test/mjsunit/es6/debug-promise-events.js
+++ b/test/mjsunit/es6/debug-promise-events.js
@@ -32,6 +32,7 @@ function listener(event, exec_state, event_data, data) {
// New promise.
assertEquals("pending", event_data.promise().status());
result.push({ promise: event_data.promise().value(), status: 0 });
+ assertTrue(exec_state.frame(0).sourceLineText().indexOf("// event") > 0);
} else if (event_data.status() !== undefined) {
// Resolve/reject promise.
updatePromise(event_data.promise().value(),
@@ -43,6 +44,7 @@ function listener(event, exec_state, event_data, data) {
assertTrue(event_data.parentPromise().isPromise());
updatePromise(event_data.promise().value(),
event_data.parentPromise().value());
+ assertTrue(exec_state.frame(0).sourceLineText().indexOf("// event") > 0);
}
} catch (e) {
print(e + e.stack)
@@ -52,15 +54,15 @@ function listener(event, exec_state, event_data, data) {
Debug.setListener(listener);
-function resolver(resolve, reject) {
- resolve();
-}
+function resolver(resolve, reject) { resolve(); }
-var p1 = new Promise(resolver);
-var p2 = p1.then().then();
-var p3 = new Promise(function(resolve, reject) { reject("rejected"); });
-var p4 = p3.then();
-var p5 = p1.then();
+var p1 = new Promise(resolver); // event
+var p2 = p1.then().then(); // event
+var p3 = new Promise(function(resolve, reject) { // event
+ reject("rejected");
+});
+var p4 = p3.then(); // event
+var p5 = p1.then(); // event
function assertAsync(b, s) {
if (b) {
« no previous file with comments | « src/runtime.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698