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

Unified Diff: test/mjsunit/wasm/stack.js

Issue 2609363004: [asm.js] [wasm] Store function start position for stack check (Closed)
Patch Set: It's 2017 already :) Created 3 years, 11 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 | « test/mjsunit/wasm/asm-wasm-stack.js ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/wasm/stack.js
diff --git a/test/mjsunit/wasm/stack.js b/test/mjsunit/wasm/stack.js
index 538c10e543a0e00c12ccd43afa0ea11cfba57887..650b1436a206e1352b75e7d346589c7a947ed2fe 100644
--- a/test/mjsunit/wasm/stack.js
+++ b/test/mjsunit/wasm/stack.js
@@ -145,5 +145,13 @@ Error.prepareStackTrace = function(error, frames) {
fail("expected wasm exception");
} catch (e) {
assertEquals("Maximum call stack size exceeded", e.message, "trap reason");
+ assertTrue(e.stack.length >= 4, "expected at least 4 stack entries");
+ verifyStack(e.stack.splice(0, 4), [
+ // isWasm function line pos file
+ [ true, "recursion", 0, 0, null],
+ [ true, "recursion", 0, 3, null],
+ [ true, "recursion", 0, 3, null],
+ [ true, "recursion", 0, 3, null]
+ ]);
}
})();
« no previous file with comments | « test/mjsunit/wasm/asm-wasm-stack.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698