Test for var declarations in eval which conflict with let

src/contexts.h

 // 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.
 
 #ifndef V8_CONTEXTS_H_
 #define V8_CONTEXTS_H_
 
 #include "src/heap/heap.h"
 #include "src/objects.h"
 
 namespace v8 {
 namespace internal {
 
 
 enum ContextLookupFlags {
   FOLLOW_CONTEXT_CHAIN = 1,
   FOLLOW_PROTOTYPE_CHAIN = 2,
+  STOP_AT_DECLARATION_SCOPE = 4,
+  SKIP_WITH_CONTEXT = 8,

[adamk, 2015/10/05 17:20:36]

Can you change these to "1 << N" form now that we're growing more of them?

[Dan Ehrenberg, 2015/10/08 23:01:26]

Done

 
   DONT_FOLLOW_CHAINS = 0,
-  FOLLOW_CHAINS = FOLLOW_CONTEXT_CHAIN | FOLLOW_PROTOTYPE_CHAIN
+  FOLLOW_CHAINS = FOLLOW_CONTEXT_CHAIN | FOLLOW_PROTOTYPE_CHAIN,
+  LEXICAL_TEST =
+      FOLLOW_CONTEXT_CHAIN | STOP_AT_DECLARATION_SCOPE | SKIP_WITH_CONTEXT,
 };
 
 
 // ES5 10.2 defines lexical environments with mutable and immutable bindings.
 // Immutable bindings have two states, initialized and uninitialized, and
 // their state is changed by the InitializeImmutableBinding method. The
 // BindingFlags enum represents information if a binding has definitely been
 // initialized. A mutable binding does not need to be checked and thus has
 // the BindingFlag MUTABLE_IS_INITIALIZED.
 //
@@ skipping 516 matching lines @@
 #endif
 
   STATIC_ASSERT(kHeaderSize == Internals::kContextHeaderSize);
   STATIC_ASSERT(EMBEDDER_DATA_INDEX == Internals::kContextEmbedderDataIndex);
 };
 
 }  // namespace internal
 }  // namespace v8
 
 #endif  // V8_CONTEXTS_H_