Permalink
Browse files

DFL-3675: Setting breakpoints from JS source tooltip is broken, sets …

…them in the wrong place
  • Loading branch information...
1 parent 6d79eed commit a4bf12ad82f519e0922fc17d2499c92ebc8cd554 Daniel Herzog committed Oct 29, 2012
Showing with 2 additions and 2 deletions.
  1. +2 −2 src/ecma-debugger/action_handler.js
@@ -184,8 +184,8 @@ window.eventHandlers.click['set-break-point'] = function(event)
var span = li.querySelector(".line-number");
line_number = span && Number(span.textContent);
}
- var script_id = window.views.js_source.getCurrentScriptId() ||
- Number(target.get_ancestor_attr("data-script-id"));
+ var script_id = Number(target.get_ancestor_attr("data-script-id")) ||
+ window.views.js_source.getCurrentScriptId();
if (script_id && line_number)
{
if (bps.script_has_breakpoint_on_line(script_id, line_number))

0 comments on commit a4bf12a

Please sign in to comment.