Closed Bug 908337 Opened 6 years ago Closed 6 years ago

add getFrameLocation(frame) function and use it wherever we get frame locations in the debugger server

Categories

(DevTools :: Debugger, defect, P3)

x86
macOS
defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 26

People

(Reporter: fitzgen, Assigned: fitzgen)

References

Details

Attachments

(1 file, 1 obsolete file)

We repeat the logic for getting a frame's url, line, and column all over the place in the debugger server. We should abstract and formalize it in a helper function.
Assignee: nobody → nfitzgerald
Status: NEW → ASSIGNED
Attachment #794170 - Flags: review?(jimb)
Attachment #794170 - Flags: review?(jimb) → review+
Priority: -- → P3
The rebased patch that landed.
Attachment #794170 - Attachment is obsolete: true
https://hg.mozilla.org/mozilla-central/rev/5f00d7c95791
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 26
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.