[parser] Skipping inner funcs: produce correct data for var redeclarations.

src/ast/scopes.cc

 // Copyright 2012 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/ast/scopes.h"
 
 #include <set>
 
 #include "src/accessors.h"
 #include "src/ast/ast.h"
@@ skipping 1065 matching lines @@
   // catch scopes: Parser::RewriteCatchPattern bypasses DeclareVariable by
   // calling DeclareLocal directly, and it doesn't make sense to add a similar
   // bypass mechanism for PreParser.
   DCHECK(is_declaration_scope() || (IsLexicalVariableMode(mode) &&
                                     (is_block_scope() || is_catch_scope())));
   DCHECK(scope_info_.is_null());
 
   // Declare the variable in the declaration scope.
   if (FLAG_preparser_scope_analysis) {
     Variable* var = LookupLocal(name);
+    DCHECK_NE(var, kDummyPreParserLexicalVariable);
+    DCHECK_NE(var, kDummyPreParserVariable);
     if (var == nullptr) {
       var = DeclareLocal(name, mode);
+    } else if (!IsLexicalVariableMode(var->mode()) &&
+               !IsLexicalVariableMode(mode)) {
+      DCHECK_EQ(mode, VAR);
+      var->set_maybe_assigned();
     }
     var->set_is_used();
   } else {
     variables_.DeclareName(zone(), name, mode);
   }
 }
 
 VariableProxy* Scope::NewUnresolved(AstNodeFactory* factory,
                                     const AstRawString* name,
                                     int start_position, VariableKind kind) {
@@ skipping 1103 matching lines @@
   Variable* function =
       is_function_scope() ? AsDeclarationScope()->function_var() : nullptr;
   bool is_function_var_in_context =
       function != nullptr && function->IsContextSlot();
   return num_heap_slots() - Context::MIN_CONTEXT_SLOTS -
          (is_function_var_in_context ? 1 : 0);
 }
 
 }  // namespace internal
 }  // namespace v8