Fix yield inside with

src/scopes.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 708 matching lines @@
 
 bool Scope::AllowsLazyCompilationWithoutContext() const {
   return !force_eager_compilation_ && HasTrivialOuterContext();
 }
 
 
 int Scope::ContextChainLength(Scope* scope) {
   int n = 0;
   for (Scope* s = this; s != scope; s = s->outer_scope_) {
     ASSERT(s != NULL);  // scope must be in the scope chain
-    if (s->num_heap_slots() > 0) n++;
+    if (s->is_with_scope() || s->num_heap_slots() > 0) n++;

[Michael Starzinger, 2013/04/26 11:31:10]

As discussed offline: It is confusing that we don't check for catch-scopes here,
as they are also part of the scope chain. It seems catch-scopes are implicitly
accounted for because they always have heap-slots. So I think we should add an
assert in that regard here.

ASSERT(!s->is_catch_scope() || s->num_heap_slots() > 0);

   }
   return n;
 }
 
 
 Scope* Scope::GlobalScope() {
   Scope* scope = this;
   while (!scope->is_global_scope()) {
     scope = scope->outer_scope();
   }
@@ skipping 650 matching lines @@
 }
 
 
 int Scope::ContextLocalCount() const {
   if (num_heap_slots() == 0) return 0;
   return num_heap_slots() - Context::MIN_CONTEXT_SLOTS -
       (function_ != NULL && function_->proxy()->var()->IsContextSlot() ? 1 : 0);
 }
 
 } }  // namespace v8::internal