Closed Bug 579571 Opened 10 years ago Closed 9 years ago

nsITimelineService should be taken out and shot

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: sfink, Assigned: matjk7)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 3 obsolete files)

It's unused and unmaintained, and only serves to trap the clueless (yes, that would be me.)

See also bug 353655 and bug 207356.

(Remember to nuke the users in tools/performance/startup/ along with it.)
Attached patch patch (obsolete) — Splinter Review
Attachment #530983 - Flags: review?(benjamin)
Attachment #530983 - Flags: review?(benjamin) → review?(tglek)
Review ping.
Comment on attachment 530983 [details] [diff] [review]
patch

Sorry for delay. Looks good
Attachment #530983 - Flags: review?(tglek) → review+
Attached patch patch for checkin (obsolete) — Splinter Review
Updated to tip and added commit message.
Attachment #530983 - Attachment is obsolete: true
Attachment #538171 - Flags: review+
Attachment #538171 - Flags: checkin?
Attached patch patch for checkin (obsolete) — Splinter Review
Updated again.
Attachment #538171 - Attachment is obsolete: true
Attachment #538728 - Flags: review+
Attachment #538728 - Flags: checkin?
Attachment #538171 - Flags: checkin?
Updated again. I would appreciate if someone could commit this!
Attachment #538728 - Attachment is obsolete: true
Attachment #546378 - Flags: review+
Attachment #546378 - Flags: checkin?
Attachment #538728 - Flags: checkin?
I think people may still be using checkin-needed keyword searches rather than combined keyword/attachment based searches. Setting checkin-needed to hopefully prevent you from having to unbitrot yet again! :-)
Assignee: nobody → matjk7
Status: NEW → ASSIGNED
Keywords: checkin-needed
OS: Linux → All
Hardware: x86_64 → All
Attachment #546378 - Flags: checkin? → checkin+
http://hg.mozilla.org/mozilla-central/rev/8ec9937d4cb6
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Keywords: dev-doc-needed
Target Milestone: --- → mozilla8
This broke comm-central when it landed into m-c.

We would appreciate a heads up on forthcoming public interface removals like this, for future. (a bug, newsgroup post, etc.)

http://hg.mozilla.org/comm-central/rev/3d95e40b74f2 as bustage fix. (we have a few js uses of this to cleanup remaining, but they are all conditional)
Depends on: 479006
This is mentioned on Firefox 8 for developers; it was never documented, so that's all that's needed here. We do need to document the newer timeline service, however.
You need to log in before you can comment on or make changes to this bug.