Closed
Bug 579571
Opened 14 years ago
Closed 13 years ago
nsITimelineService should be taken out and shot
Categories
(Core :: XPCOM, defect)
Core
XPCOM
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)
(deleted),
patch
|
matjk7
:
review+
Ms2ger
:
checkin+
|
Details | Diff | Splinter Review |
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.)
Blocks: deCOM
Assignee | ||
Comment 1•13 years ago
|
||
Attachment #530983 -
Flags: review?(benjamin)
Updated•13 years ago
|
Attachment #530983 -
Flags: review?(benjamin) → review?(tglek)
Assignee | ||
Comment 2•13 years ago
|
||
Review ping.
Comment 3•13 years ago
|
||
Comment on attachment 530983 [details] [diff] [review]
patch
Sorry for delay. Looks good
Attachment #530983 -
Flags: review?(tglek) → review+
Assignee | ||
Comment 4•13 years ago
|
||
Updated to tip and added commit message.
Attachment #530983 -
Attachment is obsolete: true
Attachment #538171 -
Flags: review+
Attachment #538171 -
Flags: checkin?
Assignee | ||
Comment 5•13 years ago
|
||
Updated again.
Attachment #538171 -
Attachment is obsolete: true
Attachment #538728 -
Flags: review+
Attachment #538728 -
Flags: checkin?
Attachment #538171 -
Flags: checkin?
Assignee | ||
Comment 6•13 years ago
|
||
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?
Comment 7•13 years ago
|
||
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
Comment 8•13 years ago
|
||
Flags: in-testsuite-
Keywords: checkin-needed
Updated•13 years ago
|
Attachment #546378 -
Flags: checkin? → checkin+
Comment 9•13 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•13 years ago
|
Keywords: dev-doc-needed
Target Milestone: --- → mozilla8
Comment 10•13 years ago
|
||
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)
Comment 11•13 years ago
|
||
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.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in
before you can comment on or make changes to this bug.
Description
•