I think it's better to leave toggleBreakpoint to be implemented in the IDE code that handles moving breakpoints with text and mouse clicks, since that code needs to get breakpoint at a row anyway, to handle cases like disabled breakpoint, condition etc. see https://github.com/ajaxorg/cloud9/blob/master/plugins-client/ext.debugger/breakpoints.js#L148