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

Unified Diff: src/runtime/runtime-debug.cc

Issue 2651043004: [wasm] Add inspector test for stepping (Closed)
Patch Set: Add expected output 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 | « no previous file | src/runtime/runtime-wasm.cc » ('j') | src/runtime/runtime-wasm.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/runtime/runtime-debug.cc
diff --git a/src/runtime/runtime-debug.cc b/src/runtime/runtime-debug.cc
index 8d1e92d1e0721b4a8e1005a84bb4e8b7bdb158d2..fd142551ab05622f47b7e360288264640fa337ed 100644
--- a/src/runtime/runtime-debug.cc
+++ b/src/runtime/runtime-debug.cc
@@ -1666,7 +1666,8 @@ namespace {
int ScriptLinePositionWithOffset(Handle<Script> script, int line, int offset) {
if (line < 0 || offset < 0) return -1;
- if (line == 0) return ScriptLinePosition(script, line) + offset;
+ if (line == 0 || offset == 0)
+ return ScriptLinePosition(script, line) + offset;
Script::PositionInfo info;
if (!Script::GetPositionInfo(script, offset, &info, Script::NO_OFFSET)) {
« no previous file with comments | « no previous file | src/runtime/runtime-wasm.cc » ('j') | src/runtime/runtime-wasm.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698