Last Comment Bug 579571 - nsITimelineService should be taken out and shot
: nsITimelineService should be taken out and shot
Status: RESOLVED FIXED
: dev-doc-complete
Product: Core
Classification: Components
Component: XPCOM (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla8
Assigned To: Matheus Kerschbaum
:
: Nathan Froyd [:froydnj]
Mentors:
Depends on: 479006
Blocks: deCOM
  Show dependency treegraph
 
Reported: 2010-07-16 16:23 PDT by Steve Fink [:sfink] [:s:]
Modified: 2011-09-30 13:36 PDT (History)
8 users (show)
Ms2ger: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---


Attachments
patch (96.73 KB, patch)
2011-05-08 22:18 PDT, Matheus Kerschbaum
taras.mozilla: review+
Details | Diff | Splinter Review
patch for checkin (96.26 KB, patch)
2011-06-08 22:02 PDT, Matheus Kerschbaum
matjk7: review+
Details | Diff | Splinter Review
patch for checkin (97.29 KB, patch)
2011-06-11 17:04 PDT, Matheus Kerschbaum
matjk7: review+
Details | Diff | Splinter Review
patch for checkin (97.53 KB, patch)
2011-07-16 20:57 PDT, Matheus Kerschbaum
matjk7: review+
Ms2ger: checkin+
Details | Diff | Splinter Review

Description Steve Fink [:sfink] [:s:] 2010-07-16 16:23:13 PDT
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.)
Comment 1 Matheus Kerschbaum 2011-05-08 22:18:26 PDT
Created attachment 530983 [details] [diff] [review]
patch
Comment 2 Matheus Kerschbaum 2011-06-08 16:15:32 PDT
Review ping.
Comment 3 (dormant account) 2011-06-08 16:19:54 PDT
Comment on attachment 530983 [details] [diff] [review]
patch

Sorry for delay. Looks good
Comment 4 Matheus Kerschbaum 2011-06-08 22:02:17 PDT
Created attachment 538171 [details] [diff] [review]
patch for checkin

Updated to tip and added commit message.
Comment 5 Matheus Kerschbaum 2011-06-11 17:04:46 PDT
Created attachment 538728 [details] [diff] [review]
patch for checkin

Updated again.
Comment 6 Matheus Kerschbaum 2011-07-16 20:57:37 PDT
Created attachment 546378 [details] [diff] [review]
patch for checkin

Updated again. I would appreciate if someone could commit this!
Comment 7 Ed Morley [:emorley] 2011-07-17 02:34:33 PDT
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! :-)
Comment 9 Joe Drew (not getting mail) 2011-07-17 17:35:19 PDT
http://hg.mozilla.org/mozilla-central/rev/8ec9937d4cb6
Comment 10 Justin Wood (:Callek) 2011-07-17 23:04:17 PDT
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 Eric Shepherd [:sheppy] 2011-09-30 13:36:36 PDT
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.

Note You need to log in before you can comment on or make changes to this bug.