Debugger: deduplicate shared function info when setting script break points.

src/factory.cc

Index: src/factory.cc
diff --git a/src/factory.cc b/src/factory.cc
index 5dac6a1ada3454980e348caf6a76b8268d51ee0d..cca3fd2213007c8e82525e913cbb9bb83bb9d118 100644
--- a/src/factory.cc
+++ b/src/factory.cc
@@ -1354,7 +1354,7 @@ Handle<JSObject> Factory::NewFunctionPrototype(Handle<JSFunction> function) {
 static bool ShouldOptimizeNewClosure(Isolate* isolate,
                                      Handle<SharedFunctionInfo> info) {
   return !info->is_toplevel() && info->is_compiled() &&
-         info->allows_lazy_compilation();
+         info->allows_lazy_compilation() && !info->optimization_disabled();

[Michael Starzinger, 2015/03/25 13:44:08]

Is this additional part of the predicate necessary for correctness? I am not a
big fan of this early checks throughout the code and would like to consolidate
those in compiler.cc as much as possible.

[Yang, 2015/03/25 14:09:42]

Done.

 }