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 User image 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 User image Matheus Kerschbaum 2011-05-08 22:18:26 PDT
Created attachment 530983 [details] [diff] [review]
patch
Comment 2 User image Matheus Kerschbaum 2011-06-08 16:15:32 PDT
Review ping.
Comment 3 User image (dormant account) 2011-06-08 16:19:54 PDT
Comment on attachment 530983 [details] [diff] [review]
patch

Sorry for delay. Looks good
Comment 4 User image 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 User image Matheus Kerschbaum 2011-06-11 17:04:46 PDT
Created attachment 538728 [details] [diff] [review]
patch for checkin

Updated again.
Comment 6 User image 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 User image 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 User image Joe Drew (not getting mail) 2011-07-17 17:35:19 PDT
http://hg.mozilla.org/mozilla-central/rev/8ec9937d4cb6
Comment 10 User image Justin Wood (:Callek) [away until Feb 27] 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 User image 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.