Closed Bug 579571 Opened 14 years ago Closed 13 years ago

nsITimelineService should be taken out and shot

Categories

(Core :: XPCOM, defect)

Product:
Core
Component:
XPCOM
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla8

People

(Reporter: sfink, Assigned: matjk7)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file, 3 obsolete files)

patch for checkin
97.53 KB, 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.)
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: 13 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.
Keywords: dev-doc-needed → dev-doc-complete

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: