Store breakpoint id instead of breakpoint where possible

Matt Loring · Matt Loring · commit 82bb2b89f161 · 2016-01-21T14:37:37.000-08:00
diff --git a/lib/debuglet.js b/lib/debuglet.js
@@ -68,7 +68,7 @@ function Debuglet(config, logger) {
   /** @private {Object.<string, Breakpoint>} */
   this.activeBreakpointMap_ = {};
 
-  /** @private {Object.<string, Breakpoint>} */
+  /** @private {Object.<string, Boolean>} */
   this.completedBreakpointMap_ = {};
 
   EventEmitter.call(this);
@@ -347,9 +347,7 @@ Debuglet.prototype.completeBreakpoint_ = function(breakpoint) {
     if (err) {
       that.logger_.error('Unable to complete breakpoint on server', err);
     } else {
-      // TODO(ofrobots): breakpoint may be holding on to a lot of data. We only
-      // need to remember the breakpoint id.
-      that.completedBreakpointMap_[breakpoint.id] = breakpoint;
+      that.completedBreakpointMap_[breakpoint.id] = true;
       that.removeBreakpoint_(breakpoint);
     }
   });
