[parser] Skipping inner funcs: produce the same scopes / variables for sloppy block funcs.

test/cctest/test-parsing.cc

 // Copyright 2012 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 9314 matching lines @@
       {"", "var var1; function f1() { eval(''); }"},
       {"", "let var1; eval('');"},
       {"", "let var1; function f1() { eval(''); }"},
       {"", "const var1 = 10; eval('');"},
       {"", "const var1 = 10; function f1() { eval(''); }"},
 
       {"", "for (var var1 = 0; var1 < 10; ++var1) { }"},
       {"", "for (let var1 = 0; var1 < 10; ++var1) { }"},
       {"", "for (const var1 = 0; var1 < 10; ++var1) { }"},
 
-      // FIXME(marja): make the corresponding cases work when foo is a sloppy
-      // block function.
+      {"",
+       "for (var var1 = 0; var1 < 10; ++var1) { function foo() { var1; } }"},
+      {"",
+       "for (let var1 = 0; var1 < 10; ++var1) { function foo() { var1; } }"},
+      {"",
+       "for (const var1 = 0; var1 < 10; ++var1) { function foo() { var1; } }"},
       {"",
        "'use strict'; for (var var1 = 0; var1 < 10; ++var1) { function foo() { "
        "var1; } }"},
       {"",
        "'use strict'; for (let var1 = 0; var1 < 10; ++var1) { function foo() { "
        "var1; } }"},
       {"",
        "'use strict'; for (const var1 = 0; var1 < 10; ++var1) { function foo() "
        "{ var1; } }"},
+
+      {"", "if (true) { function f1() {} }"},
+      {"", "if (true) { function f1() {} function f1() {} }"},
+      {"", "if (true) { if (true) { function f1() {} } }"},
+      {"", "if (true) { if (true) { function f1() {} function f1() {} } }"},
+      {"", "if (true) { function f1() {} f1 = 3; }"},
+
+      {"", "if (true) { function f1() {} function foo() { f1; } }"},
+      {"", "if (true) { function f1() {} } function foo() { f1; }"},
+      {"",
+       "if (true) { function f1() {} function f1() {} function foo() { f1; } "
+       "}"},
+      {"",
+       "if (true) { function f1() {} function f1() {} } function foo() { f1; "
+       "}"},
+      {"",
+       "if (true) { if (true) { function f1() {} } function foo() { f1; } }"},
+      {"",
+       "if (true) { if (true) { function f1() {} function f1() {} } function "
+       "foo() { f1; } }"},
+      {"", "if (true) { function f1() {} f1 = 3; function foo() { f1; } }"},
+      {"", "if (true) { function f1() {} f1 = 3; } function foo() { f1; }"},
+
+      {"", "function inner2() { if (true) { function f1() {} } }"},
+      {"", "function inner2() { if (true) { function f1() {} f1 = 3; } }"},
+
+      {"", "var f1 = 1; if (true) { function f1() {} }"},
+      {"", "var f1 = 1; if (true) { function f1() {} } function foo() { f1; }"},
   };
 
   for (unsigned i = 0; i < arraysize(inners); ++i) {
     // First compile with the lazy inner function and extract the scope data.
     const char* inner_function = lazy_inner;
     int inner_function_len = Utf8LengthHelper(inner_function) - 4;
 
     int params_len = Utf8LengthHelper(inners[i].params);
     int source_len = Utf8LengthHelper(inners[i].source);
     int len =
@@ skipping 49 matching lines @@
         eager_info.literal()->scope()->inner_scope()->inner_scope();
     DCHECK_NOT_NULL(scope);
     DCHECK_NULL(scope->sibling());
     DCHECK(scope->is_function_scope());
 
     size_t index = 0;
     i::ScopeTestHelper::CompareScopeToData(
         scope, lazy_info.preparsed_scope_data(), index);
   }
 }