Chromium Code Reviews| Index: test/mjsunit/debug-function-scopes.js |
| diff --git a/test/mjsunit/debug-function-scopes.js b/test/mjsunit/debug-function-scopes.js |
| index b51e8b4432b3fb53f585d8f43b7a658eff176994..399106b7c09579ab3549148c1584625b7cb7b49e 100644 |
| --- a/test/mjsunit/debug-function-scopes.js |
| +++ b/test/mjsunit/debug-function-scopes.js |
| @@ -44,11 +44,12 @@ function CheckScope(scope_mirror, scope_expectations, expected_scope_type) { |
| // A copy of the scope types from mirror-debugger.js. |
| var ScopeType = { Global: 0, |
| - Local: 1, |
| - With: 2, |
| - Closure: 3, |
| - Catch: 4, |
| - Block: 5 }; |
| + Script: 1, |
|
aandrey
2014/11/14 15:58:47
please update this also
Dmitry Lomov (no reviews)
2014/11/14 16:37:27
All updated in patch set 3
|
| + Local: 2, |
| + With: 3, |
| + Closure: 4, |
| + Catch: 5, |
| + Block: 6 }; |
| var f1 = (function F1(x) { |
| function F2(y) { |
| @@ -68,21 +69,23 @@ var f1 = (function F1(x) { |
| var mirror = Debug.MakeMirror(f1); |
| -assertEquals(5, mirror.scopeCount()); |
| +assertEquals(6, mirror.scopeCount()); |
| CheckScope(mirror.scope(0), { a: 4, b: 5 }, ScopeType.Closure); |
| CheckScope(mirror.scope(1), { w: 5, v: "Capybara" }, ScopeType.With); |
| CheckScope(mirror.scope(2), { y: 17, z: 22 }, ScopeType.Closure); |
| CheckScope(mirror.scope(3), { x: 5 }, ScopeType.Closure); |
| -CheckScope(mirror.scope(4), {}, ScopeType.Global); |
| +CheckScope(mirror.scope(4), {}, ScopeType.Script); |
| +CheckScope(mirror.scope(5), {}, ScopeType.Global); |
| var f2 = function() { return 5; } |
| var mirror = Debug.MakeMirror(f2); |
| -assertEquals(1, mirror.scopeCount()); |
| +assertEquals(2, mirror.scopeCount()); |
| -CheckScope(mirror.scope(0), {}, ScopeType.Global); |
| +CheckScope(mirror.scope(0), {}, ScopeType.Script); |
| +CheckScope(mirror.scope(1), {}, ScopeType.Global); |
| var f3 = (function F1(invisible_parameter) { |
| var invisible1 = 1; |
| @@ -99,11 +102,12 @@ var f3 = (function F1(invisible_parameter) { |
| var mirror = Debug.MakeMirror(f3); |
| -assertEquals(3, mirror.scopeCount()); |
| +assertEquals(4, mirror.scopeCount()); |
| CheckScope(mirror.scope(0), { visible2: 20 }, ScopeType.Closure); |
| CheckScope(mirror.scope(1), { visible1: 10 }, ScopeType.Closure); |
| -CheckScope(mirror.scope(2), {}, ScopeType.Global); |
| +CheckScope(mirror.scope(2), {}, ScopeType.Script); |
| +CheckScope(mirror.scope(3), {}, ScopeType.Global); |
| var f4 = (function One() { |
| @@ -122,11 +126,12 @@ var f4 = (function One() { |
| var mirror = Debug.MakeMirror(f4); |
| -assertEquals(3, mirror.scopeCount()); |
| +assertEquals(4, mirror.scopeCount()); |
| CheckScope(mirror.scope(0), { e2: "I'm error 2" }, ScopeType.Catch); |
| CheckScope(mirror.scope(1), { e1: "I'm error 1" }, ScopeType.Catch); |
| -CheckScope(mirror.scope(2), {}, ScopeType.Global); |
| +CheckScope(mirror.scope(2), {}, ScopeType.Script); |
| +CheckScope(mirror.scope(3), {}, ScopeType.Global); |
| var f5 = (function Raz(p1, p2) { |
| @@ -141,11 +146,12 @@ var f5 = (function Raz(p1, p2) { |
| var mirror = Debug.MakeMirror(f5); |
| -assertEquals(3, mirror.scopeCount()); |
| +assertEquals(4, mirror.scopeCount()); |
| CheckScope(mirror.scope(0), { p4: 20, p6: 22 }, ScopeType.Closure); |
| CheckScope(mirror.scope(1), { p1: 1 }, ScopeType.Closure); |
| -CheckScope(mirror.scope(2), {}, ScopeType.Global); |
| +CheckScope(mirror.scope(2), {}, ScopeType.Script); |
| +CheckScope(mirror.scope(3), {}, ScopeType.Global); |
| function CheckNoScopeVisible(f) { |