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

Unified Diff: test/debugger/test-api.js

Issue 2515133003: [debug-wrapper] Migrate wasm/frame-inspection test (Closed)
Patch Set: Adjust expectations Created 4 years, 1 month 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/debugger/debug/wasm/frame-inspection.js ('k') | test/mjsunit/wasm/frame-inspection.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/debugger/test-api.js
diff --git a/test/debugger/test-api.js b/test/debugger/test-api.js
index 069f81796d74e411a169b10ac60ebbed79471bac..cb9022b0c04c0dd48212e2302aaaa9b77d6305ff 100644
--- a/test/debugger/test-api.js
+++ b/test/debugger/test-api.js
@@ -639,8 +639,8 @@ class DebugWrapper {
return scopes;
};
- return { sourceColumn : () => loc.column,
- sourceLine : () => loc.line + 1,
+ return { sourceColumn : () => column,
+ sourceLine : () => line + 1,
sourceLineText : () => loc.sourceText,
evaluate : (expr) => this.evaluateOnCallFrame(frame, expr),
functionName : () => frame.functionName,
@@ -693,7 +693,7 @@ class DebugWrapper {
return { id : () => id,
name : () => name,
- source : () => this.eventDataScriptSource(id),
+ source : () => this.eventDataScriptSource(params.scriptId),
setSource : (src) => this.eventDataScriptSetSource(id, src)
};
}
« no previous file with comments | « test/debugger/debug/wasm/frame-inspection.js ('k') | test/mjsunit/wasm/frame-inspection.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698