The default bug view has changed. See this FAQ.

nsITimelineService should be taken out and shot

RESOLVED FIXED in mozilla8

Status

()

Core
XPCOM
RESOLVED FIXED
7 years ago
6 years ago

People

(Reporter: sfink, Assigned: Matheus Kerschbaum)

Tracking

(Depends on: 1 bug, Blocks: 1 bug, {dev-doc-complete})

Trunk
mozilla8
dev-doc-complete
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 3 obsolete attachments)

97.53 KB, patch
Matheus Kerschbaum
: review+
Ms2ger
: checkin+
Details | Diff | Splinter Review
(Reporter)

Description

7 years ago
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: 105431
(Assignee)

Comment 1

6 years ago
Created attachment 530983 [details] [diff] [review]
patch
Attachment #530983 - Flags: review?(benjamin)
Attachment #530983 - Flags: review?(benjamin) → review?(tglek)
(Assignee)

Comment 2

6 years ago
Review ping.

Comment 3

6 years ago
Comment on attachment 530983 [details] [diff] [review]
patch

Sorry for delay. Looks good
Attachment #530983 - Flags: review?(tglek) → review+
(Assignee)

Comment 4

6 years ago
Created attachment 538171 [details] [diff] [review]
patch for checkin

Updated to tip and added commit message.
Attachment #530983 - Attachment is obsolete: true
Attachment #538171 - Flags: review+
Attachment #538171 - Flags: checkin?
(Assignee)

Comment 5

6 years ago
Created attachment 538728 [details] [diff] [review]
patch for checkin

Updated again.
Attachment #538171 - Attachment is obsolete: true
Attachment #538728 - Flags: review+
Attachment #538728 - Flags: checkin?
Attachment #538171 - Flags: checkin?
(Assignee)

Comment 6

6 years ago
Created attachment 546378 [details] [diff] [review]
patch for 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
http://hg.mozilla.org/integration/mozilla-inbound/rev/8ec9937d4cb6
Flags: in-testsuite-
Keywords: checkin-needed

Updated

6 years ago
Attachment #546378 - Flags: checkin? → checkin+
http://hg.mozilla.org/mozilla-central/rev/8ec9937d4cb6
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Updated

6 years ago
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.
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.