Description Michael Rennie 2010-05-03 14:31:33 EDT

Created attachment 166829 [details]
fix

code from HEAD

using the following test script:

var func1 = func1 || {};
func1.fib = function f1(n) {};
var func2 = function f2(n) {};
function f3(n){};

trying to toggle function breakpoints with the cursor to the LHS of the assignment operator will fail in the editor, because we are trying to resolve the JS element at the cursor locale, which might not be what we want. The better solution would be to compute the start position while visiting the AST to find the valid locale for the breakpoint.

This would make a good RC1 fix, as it makes toggling a function breakpoint behave as expected.