[compiler] Collect eager inner functions for compilation during renumbering.

test/unittests/compiler-dispatcher/compiler-dispatcher-job-unittest.cc

 // Copyright 2016 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 <memory>
 
 #include "include/v8.h"
 #include "src/api.h"
 #include "src/ast/ast.h"
 #include "src/ast/scopes.h"
@@ skipping 304 matching lines @@
   V8::GetCurrentPlatform()->CallOnBackgroundThread(background_task,
                                                    Platform::kShortRunningTask);
   semaphore.Wait();
   ASSERT_TRUE(job->FinalizeCompilingOnMainThread());
   ASSERT_TRUE(job->status() == CompileJobStatus::kDone);
 
   job->ResetOnMainThread();
   ASSERT_TRUE(job->status() == CompileJobStatus::kInitial);
 }
 
-TEST_F(CompilerDispatcherJobTest, LazyInnerFunctions) {
-  const char script[] =
-      "function g() {\n"
-      "  f = function() {\n"
-      "    e = (function() { return 42; });\n"
-      "    return e;\n"
-      "  };\n"
-      "  return f;\n"
-      "}\n"
-      "g();";
-  Handle<JSFunction> f = Handle<JSFunction>::cast(RunJS(isolate(), script));
-
-  std::unique_ptr<CompilerDispatcherJob> job(new CompilerDispatcherJob(
-      i_isolate(), tracer(), handle(f->shared()), FLAG_stack_size));
-
-  job->PrepareToParseOnMainThread();
-  job->Parse();
-  ASSERT_TRUE(job->FinalizeParsingOnMainThread());
-  ASSERT_TRUE(job->PrepareToCompileOnMainThread());
-  job->Compile();
-  ASSERT_TRUE(job->FinalizeCompilingOnMainThread());
-  ASSERT_TRUE(job->status() == CompileJobStatus::kDone);
-
-  Handle<JSFunction> e = Handle<JSFunction>::cast(RunJS(isolate(), "f();"));
-
-  ASSERT_FALSE(e->shared()->HasBaselineCode());

[marja, 2017/01/05 19:25:22]

Hmm, I don't understand why this test is removed.. is something it asserts now
false?

[rmcilroy, 2017/01/06 17:08:14]

This was part of the revert of the LazyCompileMode cl which introduced this
test, but you are right, the constraints are still the same and this test should
still pass, so adding it back.

-
-  job->ResetOnMainThread();
-  ASSERT_TRUE(job->status() == CompileJobStatus::kInitial);
-}
-
 }  // namespace internal
 }  // namespace v8