Parse harmony let declarations.

test/mjsunit/harmony/debug-blockscopes.js

Index: test/mjsunit/harmony/debug-blockscopes.js
diff --git a/test/mjsunit/harmony/debug-blockscopes.js b/test/mjsunit/harmony/debug-blockscopes.js
index 30f681f129da92cdc6a97fea1cd0bb26c1e91a23..b21da37bc259f5068e105aff17d45a146a4baaf3 100644
--- a/test/mjsunit/harmony/debug-blockscopes.js
+++ b/test/mjsunit/harmony/debug-blockscopes.js
@@ -157,7 +157,6 @@ function CheckScopeContent(content, number, exec_state) {
     scope_size--;
   }
 
-  // TODO(keuchel): print('count' + count + ' scopesize' + scope_size);
   if (count != scope_size) {
     print('Names found in scope:');
     var names = scope.scopeObject().propertyNames();
@@ -236,7 +235,7 @@ EndTest();
 BeginTest("Local 3");
 
 function local_3(a) {
-  var x = 3;
+  let x = 3;
   debugger;
 }
 
@@ -253,8 +252,8 @@ EndTest();
 BeginTest("Local 4");
 
 function local_4(a, b) {
-  var x = 3;
-  var y = 4;
+  let x = 3;
+  let y = 4;
   debugger;
 }
 
@@ -361,29 +360,34 @@ with_block_4();
 EndTest();
 
 
-// TODO(keuchel):
 // Simple closure formed by returning an inner function referering to an outer
 // block local variable and an outer function's parameter.
-// BeginTest("Closure 1");
-//
-// function closure_1(a) {
-//   {
-//     let x = 3;
-//     function f() {
-//       debugger;
-//       return a + x;
-//     };
-//   }
-//   return f;
-// }
-//
-// listener_delegate = function(exec_state) {
-//   CheckScopeChain([debug.ScopeType.Local,
-//                    debug.ScopeType.Closure,
-//                    debug.ScopeType.Global], exec_state);
-//   // CheckScopeContent({}, 0, exec_state);
-//   // CheckScopeContent({}, 1, exec_state);
-//   // CheckScopeContent({a:1}, 2, exec_state);
-// };
-// closure_1(1)();
-// EndTest();
+BeginTest("Closure 1");
+
+function closure_1(a) {
+  var x = 2;
+  let y = 3;
+  if (true) {
+    let z = 4;
+    function f() {
+      debugger;
+      return a + x + y + z;
+    };
+    return f;
+  }
+}
+
+listener_delegate = function(exec_state) {
+  CheckScopeChain([debug.ScopeType.Local,
+                   debug.ScopeType.Block,
+                   debug.ScopeType.Closure,
+                   debug.ScopeType.Global], exec_state);
+  CheckScopeContent({}, 0, exec_state);
+  // TODO(keuchel): block local functions in scope contents
+  // CheckScopeContent({z:4}, 1, exec_state);

[Lasse Reichstein, 2011/08/12 08:08:33]

Remove the TODO and commented code.
If it's not working, file a bug for it and put a test in mjsunit/bugs (those are
assumed to fail).

[Steven, 2011/08/16 09:05:32]

Done.

+  CheckScopeContent({a:1,x:2,y:3}, 2, exec_state);
+};
+closure_1(1)();
+EndTest();
+
+