Closed Bug 1258006 Opened 9 years ago Closed 6 years ago

Add test for edge case with top-level sourcemapped breakpoints

Categories

(DevTools :: Debugger, defect, P3)

Product:
DevTools
Component:
Debugger
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: jlong, Unassigned)

References

(Blocks 1 open bug)

Details

This code: https://github.com/mozilla/gecko-dev/blob/master/devtools/server/actors/script.js#L1966-L1972 We had a regression that made that `unsafeSynchronize` not run, and we never noticed. Top-level sourcemapped breakpoints didn't work sometimes then. This is fixed in bug 1177329 but we still have no tests for it. We should write a test that fails when we hit whatever condition that that code solves. I'm not familiar with the problem enough to write a test yet.
Priority: -- → P2
Blocks: source-maps
Product: Firefox → DevTools
Priority: P2 → P3

We now handle sourcemaps in the client, so this is no longer relevant.

Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.