[debugger] simplify compile for debugging.

src/compiler.cc

Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index 9fbf795e4923cc8858adced3f84acdb7d3620427..164f23815a02c8859dc18fe200f71f506f6ac807 100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -1120,12 +1120,10 @@ bool Compiler::CompileOptimized(Handle<JSFunction> function,
   return true;
 }
 
-bool Compiler::CompileDebugCode(Handle<JSFunction> function) {
-  return CompileDebugCode(handle(function->shared()));
-}
-
 bool Compiler::CompileDebugCode(Handle<SharedFunctionInfo> shared) {
   Isolate* isolate = shared->GetIsolate();
+  // Code that cannot be compiled lazily are internal and not debuggable.
+  DCHECK(shared->allows_lazy_compilation());

[Michael Starzinger, 2016/11/14 09:19:11]

I don't fully understand why this became a pre-condition for this method. Can we
drop it? If this really is an important pre-condition we should be checking then
we should find a better place for it so that all paths benefit from it. Can we
drop it for now?

   DCHECK(AllowCompilation::IsAllowed(isolate));
 
   // Start a compilation.