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

Unified Diff: test/mjsunit/wasm/debug-disassembly.js

Issue 2304943002: Do not call into JS from WasmDebugInfo::GetFunctionScript (Closed)
Patch Set: Rebase Created 4 years, 3 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/wasm/wasm-debug.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/mjsunit/wasm/debug-disassembly.js
diff --git a/test/mjsunit/wasm/debug-disassembly.js b/test/mjsunit/wasm/debug-disassembly.js
index 13b93cb29bf77cd96f4e23e6b2fa4ef8b0129ba9..976098a8536ba427cc8860b06c6f659a509b2777 100644
--- a/test/mjsunit/wasm/debug-disassembly.js
+++ b/test/mjsunit/wasm/debug-disassembly.js
@@ -107,17 +107,10 @@ function setup() {
(function testRegisteredWasmScripts1() {
setup();
Debug.setListener(listener);
- // Initially 0 scripts.
- assertEquals(0, num_wasm_scripts);
// Call the "call_import" function -> 1 script.
module.exports.call_import();
- assertEquals(1, num_wasm_scripts);
- // Call "call_import" again -> still just 1 script.
module.exports.call_import();
- assertEquals(1, num_wasm_scripts);
- // Call "call_call_import" -> now 2 scripts.
module.exports.call_call_import();
- assertEquals(2, num_wasm_scripts);
Debug.setListener(null);
assertEquals(3, break_count);
@@ -127,11 +120,7 @@ function setup() {
(function testRegisteredWasmScripts2() {
setup();
Debug.setListener(listener);
- // Initially 0 scripts.
- assertEquals(0, num_wasm_scripts);
- // Call the "call_call_import" function -> 2 scripts should be registered.
module.exports.call_call_import();
- assertEquals(2, num_wasm_scripts);
Debug.setListener(null);
assertEquals(1, break_count);
« no previous file with comments | « src/wasm/wasm-debug.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698