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

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

Issue 2519773003: [debug] Add Eval scope type to inspector protocol (Closed)
Patch Set: Address comments 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/regress-5207.js ('k') | test/inspector/debugger/eval-scopes.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..5fbc4f9b69e78b1b79f28cfdf32b7f89fd82d382 100644
--- a/test/debugger/test-api.js
+++ b/test/debugger/test-api.js
@@ -391,6 +391,7 @@ class DebugWrapper {
case "catch": return this.ScopeType.Catch;
case "block": return this.ScopeType.Block;
case "script": return this.ScopeType.Script;
+ case "eval": return this.ScopeType.Eval;
default: %AbortJS("Unexpected scope type");
}
}
@@ -405,7 +406,7 @@ class DebugWrapper {
}
}
- if (found == null) return { isUndefined : true }
+ if (found == null) return { isUndefined : () => true };
const val = { value : () => found.value.value };
return { value : () => val,
« no previous file with comments | « test/debugger/debug/regress-5207.js ('k') | test/inspector/debugger/eval-scopes.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698