Various clean-ups after top-level lexical declarations are done.

test/cctest/test-decls.cc

 // Copyright 2007-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 791 matching lines @@
     const char* script1 = "(function(){ return y1 })()";
     const char* script2 = "(function(){ return y2 })()";
 
     context.Check(script1, EXPECT_EXCEPTION);
     context.Check("this.y1 = 1; this.y2 = 2; 0;",
                   EXPECT_RESULT, Number::New(isolate, 0));
     context.Check(script1,
                   EXPECT_RESULT, Number::New(isolate, 1));
     context.Check("'use strict'; let y1 = 3; 0;",
                   EXPECT_RESULT, Number::New(isolate, 0));
-    // TODO(dslomov): still returns 1 not 3
-    // context.Check(script1,
-    //               EXPECT_RESULT, Number::New(isolate, 3));
+    context.Check(script1,
+                  EXPECT_RESULT, Number::New(isolate, 3));
     context.Check("y1 = 4;",
                   EXPECT_RESULT, Number::New(isolate, 4));
-    // TODO(dslomov): still returns 1 not 4
-    // context.Check(script1,
-    //               EXPECT_RESULT, Number::New(isolate, 4));
+    context.Check(script1,
+                  EXPECT_RESULT, Number::New(isolate, 4));
 
     context.Check(script2,
                   EXPECT_RESULT, Number::New(isolate, 2));
     context.Check("'use strict'; let y2 = 5; 0;",
                   EXPECT_RESULT, Number::New(isolate, 0));
-    // TODO(dslomov): still returns 1 not 4
-    // context.Check(script1,
-    //               EXPECT_RESULT, Number::New(isolate, 4));
-    // TODO(dslomov): still returns 2 not 5
-    // context.Check(script2,
-    //              EXPECT_RESULT, Number::New(isolate, 5));
+    context.Check(script1,
+                  EXPECT_RESULT, Number::New(isolate, 4));
+    context.Check(script2,
+                  EXPECT_RESULT, Number::New(isolate, 5));
   }
 }
 
 
 TEST(GlobalLexicalOSR) {
   i::FLAG_use_strict = true;
   i::FLAG_harmony_scoping = true;
   i::FLAG_harmony_modules = true;
 
   v8::Isolate* isolate = CcTest::isolate();
@@ skipping 292 matching lines @@
                     Number::New(CcTest::isolate(), 13));
     }
     context.Check("global.x", EXPECT_RESULT,
                   Number::New(CcTest::isolate(), 20));
     context.Check("%OptimizeFunctionOnNextCall(f); f(41); x", EXPECT_RESULT,
                   Number::New(CcTest::isolate(), 41));
     context.Check("global.x", EXPECT_RESULT,
                   Number::New(CcTest::isolate(), 20));
   }
 }