956466 - Debugging with source maps highlights the wrong line when stepping in this case, incorrect line numbers

Status: RESOLVED WORKSFORME

(DevTools :: Debugger, defect, P3)

Description

[Nickolay_Ponomarev]

Attachment: the JS file for the test

In Nightly, OS X:

1. Set breakpoint at testcase.html:5
        console.log(regExp.execDebug("a"));
2. Click the button on the test page.
3. Step in
4. In RegExpJS.prototype.execDebug step over until line 202
    var res = exec(str, this.$startNode, i, this.multiline, this.ignoreCase);
5. Step into exec()

Actual results: Line 814, just above the function declaration is highlighted:
>> 814 }
   815 function exec(matchStr, startNode, lastIndex, multiline, ignoreCase) {

Expected results: the line just below the function declaration is highlighted:
   815 function exec(matchStr, startNode, lastIndex, multiline, ignoreCase) {
>> 816   idCounterTrace = 0;

Also worth noting that the line numbers are not as specified in the source map. In the source map (and the original) exec.js has 1005 lines, while the debugger mangled the source to fit into 899 lines.

The Chrome devtools both display the correct version of the file and highlight the correct line:
   913 function exec(matchStr, startNode, lastIndex, multiline, ignoreCase) {
>> 914    idCounterTrace = 0;

If I try the steps to reproduce again without reloading I always end up on line 814.
If I reload the page and try to reproduce, I end up on line 914, which would be correct, except the displayed source is still 899 lines.

Comment 1

[Nickolay_Ponomarev]

Attachment: testcase

Comment 2

[Nickolay_Ponomarev]

Attachment: mangled exec.js as the debugger displays it

Comment 3

[Nickolay_Ponomarev]

Attachment: the original exec.js, matches the version in the source map

Comment 4

[Nick Fitzgerald [:fitzgen] [⏰PST; UTC-8]]

Might be the same bug as bug 927158

Comment 5

[Eddy Bruel [:ejpbruel]]

That bug has since been fixed. I don't have time to check it right now, but can this bug still be reproduced?

Comment 6

[Nickolay_Ponomarev]

Yes, this works for me on the 2014-09-21 nightly.