Closed
Bug 1251757
Opened 9 years ago
Closed 7 years ago
Unable to break via source maps
Categories
(DevTools :: Debugger, defect, P2)
DevTools
Debugger
Tracking
(firefox47 affected)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
firefox47 | --- | affected |
People
(Reporter: yury, Unassigned)
References
(Blocks 1 open bug, )
Details
Attachments
(1 file)
(deleted),
image/png
|
Details |
STR:
1. open devtools debugger tab (check "Show Original Sources" option)
2. open http://people.mozilla.org/~ydelendik/tmp/dbgwast/t.html
3. select "labels.wast" source code
4. set breakpoint at line 12
5. refresh
Actual result:
Debugger stops but it does not show where is stops, no locals are visible
Expected result:
Debugger stops and let code to be stepped
Comment 1•9 years ago
|
||
It looks like the breakpoint you're setting is in top-level code that is executed immediately on refresh.
This is a difficult problem. We need to load the source map before we set any breakpoints, or the breakpoints won't end up where you expect. We need to set the breakpoints before the script starts executing, or control will have already passed through the point where the breakpoint was supposed to be set. However, we also don't really want to wait for the source map to be loaded before the script starts executing, since that will hang the UI.
I'd like for this to be fixed, but fixing it properly will require us to rethink some parts of the design of the debugger. That makes this a medium term project, so I'm going to flag this as P2.
Priority: -- → P2
Comment 2•7 years ago
|
||
This seems to work for me now.
Yury, would you mind giving it a try as well?
Flags: needinfo?(ydelendik)
Updated•7 years ago
|
Blocks: source-maps
Comment 3•7 years ago
|
||
Seems to work for me. However the __locals object is empty and I'm not sure if that's correct.
Reporter | ||
Comment 4•7 years ago
|
||
The test case at people.mozilla.org is lost, I'm unable to confirm the fix. Closing based on comment 2 and comment 3.
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(ydelendik)
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Firefox → DevTools
You need to log in
before you can comment on or make changes to this bug.
Description
•