fix: Display breakpoints set into the future

nodejs · Sep 8, 2016 · f2ad1ae · f2ad1ae
1 parent 2db4660
commit f2ad1ae
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/lib/node-inspect.js b/lib/node-inspect.js
@@ -515,6 +515,10 @@ function createCommandContext(inspector) {
     return new Proxy(new RemoteObject(result), createRemoteObjectProxyHandler(inspector.client));
   }
 
+  function handleBreakpointResolved({ breakpointId, location }) {
+    knownBreakpoints.set(breakpointId, location);
+  }
+
   const ctx = {
     get help() {
       const commands = [
@@ -574,7 +578,7 @@ function createCommandContext(inspector) {
     setBreakpoint(script, line, condition, silent) {
       function registerBreakpoint({ breakpointId, actualLocation }) {
         if (actualLocation) {
-          knownBreakpoints.set(breakpointId, actualLocation);
+          handleBreakpointResolved({ breakpointId, location: actualLocation });
           if (!silent) return ctx.list(5);
         } else {
           print(`Warning: script '${script}' was not loaded yet.`);
@@ -786,7 +790,7 @@ function createCommandContext(inspector) {
       .then(() => repl.displayPrompt());
   });
 
-  // TODO: Debugger.on('breakpointResolved', ({ })
+  Debugger.on('breakpointResolved', handleBreakpointResolved);
 
   Debugger.on('scriptParsed', ({ scriptId, url }) => {
     if (url) {