Set ScopeInfo when compiling for unit tests.

test/cctest/compiler/function-tester.h

 // Copyright 2014 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_CCTEST_COMPILER_FUNCTION_TESTER_H_
 #define V8_CCTEST_COMPILER_FUNCTION_TESTER_H_
 
 #include "src/v8.h"
 #include "test/cctest/cctest.h"
 
@@ skipping 29 matching lines @@
     CompilationInfoWithZone info(function);
 
     CHECK(Parser::Parse(&info));
     StrictMode strict_mode = info.function()->strict_mode();
     info.SetStrictMode(strict_mode);
     info.SetOptimizing(BailoutId::None(), Handle<Code>(function->code()));
     CHECK(Rewriter::Rewrite(&info));
     CHECK(Scope::Analyze(&info));
     CHECK_NE(NULL, info.scope());
 
+    if (!info.shared_info().is_null()) {

[sigurds, 2014/08/08 11:53:44]

Please advice if you think the null check is necessary. (Copied from
compiler.cc)

[Michael Starzinger, 2014/08/14 08:48:07]

Since we initialize the CompilationInfo from a JSFunction, the
SharedFunctionInfo should be initialized. I would be fine with dropping the
check.

[sigurds, 2014/08/14 09:53:33]

Done.

+      Handle<ScopeInfo> scope_info =
+          ScopeInfo::Create(info.scope(), info.zone());
+      info.shared_info()->set_scope_info(*scope_info);
+    }
+
     EnsureDeoptimizationSupport(&info);
 
     Pipeline pipeline(&info);
     Handle<Code> code = pipeline.GenerateCode();
 
     CHECK(!code.is_null());
     function->ReplaceCode(*code);
 #elif USE_CRANKSHAFT
     Handle<Code> unoptimized = Handle<Code>(function->code());
     Handle<Code> code = Compiler::GetOptimizedCode(function, unoptimized,
@@ skipping 125 matching lines @@
 
   Handle<Object> true_value() { return isolate->factory()->true_value(); }
 
   Handle<Object> false_value() { return isolate->factory()->false_value(); }
 };
 }
 }
 }  // namespace v8::internal::compiler
 
 #endif  // V8_CCTEST_COMPILER_FUNCTION_TESTER_H_