[inspector] fixed Debugger.getPossibleBreakpoints

src/debug/debug.cc

Index: src/debug/debug.cc
diff --git a/src/debug/debug.cc b/src/debug/debug.cc
index c0420a273fbf030439dc4471a3daa66091026e8a..919f968658e3c2bba07de3b49018dead33106e3a 100644
--- a/src/debug/debug.cc
+++ b/src/debug/debug.cc
@@ -1360,7 +1360,8 @@ bool Debug::GetPossibleBreakpoints(Handle<Script> script, int start_position,
           was_compiled = true;
         }
       }
-      if (!candidates[i]->HasDebugInfo()) CreateDebugInfo(candidates[i]);
+      if (!EnsureDebugInfo(candidates[i], Handle<JSFunction>::null()))

[Yang, 2016/11/30 20:44:36]

I think you can remove the whole if block checking for HasDebugCode.

[kozy, 2016/11/30 22:21:58]

I can remove this block if I'll pass JSFunction as second arguments of
EnsureDebugInfo, otherwise DCHECK(shared->HasDebugCode()); will fails inside of
EnsureDebugInfo. Can I get any of JSFunction object from shared function info?
And I think that it's possible that current shared function info doesn't have
any corresponded JSFunction object.

+        return false;
     }
     if (was_compiled) continue;