Closed Bug 453181 Opened 12 years ago Closed 12 years ago

Remove code that is labeled REMOVE ME

Categories

(Toolkit :: Places, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla1.9.1b1

People

(Reporter: sdwilsh, Assigned: sdwilsh)

Details

Attachments

(1 file)

We have all sorts of code that is labeled REMOVE ME with some timeline.  All those timelines have passed, so let's actually remove this code.
Attached patch v1.0Splinter Review
Attachment #336945 - Flags: review?(dietrich)
Whiteboard: [has patch][needs review dietrich]
Comment on attachment 336945 [details] [diff] [review]
v1.0

r=me
Attachment #336945 - Flags: review?(dietrich) → review+
Whiteboard: [has patch][needs review dietrich] → [has patch][has review][can land]
Blocks: 428565
This didn't apply cleanly at all when I re-ordered it in my patch queue.  I've fixed it, and pushed to mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/3025b969f961
No longer blocks: 428565
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Flags: in-testsuite-
Flags: in-litmus-
Resolution: --- → FIXED
Whiteboard: [has patch][has review][can land]
You need to log in before you can comment on or make changes to this bug.