Closed Bug 924879 Opened 7 years ago Closed 7 years ago

Minor cleanups for Parser.jsm

Categories

(DevTools :: Debugger, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 27

People

(Reporter: vporof, Assigned: vporof)

References

Details

Attachments

(1 file)

This module has some dead code that's not exercised by tests, along with some other small cruft that has accumulated. The original intent for it was bug 852931, however it hasn't landed yet and doesn't look like it will in the very near future.

Since I've been changing things in bug 725235, I'm going to remove the unused code because I don't know if I'm breaking things or not.
Priority: -- → P2
Comment on attachment 814855 [details] [diff] [review]
dbg-parser-cleanup.patch

Review of attachment 814855 [details] [diff] [review]:
-----------------------------------------------------------------

Looks fine.
Attachment #814855 - Flags: review?(past) → review+
https://hg.mozilla.org/mozilla-central/rev/a87b6c2d9151
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 27
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.