Closed Bug 277853 Opened 20 years ago Closed 20 years ago

###!!! ASSERTION: You are calling CreateInstance "{b148eed2-236d-11d3-b35c-00a0cc3c1cde}" when a service for this CID already exists!: 'Error', file r:/cenzic/mozilla/xpcom/components/nsComponentManager.cpp, line 1881

Categories

(SeaMonkey :: Bookmarks & History, defect)

Product:
SeaMonkey
Component:
Bookmarks & History
Platform:
x86
Windows XP
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
mozilla1.8beta1

People

(Reporter: timeless, Assigned: csthomas)

References

(
URL
)

Details

Attachments

(2 files)

xpfe patch
2.41 KB, patch
timeless
: review+
neil
: superreview+
Details | Diff | Splinter Review
browser (firefox) patch
2.67 KB, patch
mconnor
: review+
Details | Diff | Splinter Review 
This happens to be the NS_SOUND_CID.

Almost everyone behaves properly and gets sounds by contract id. From my quick
scan everyone except for bookmarks uses the contract id.

Note that download manager improperly uses getservice, but that's bug 277852.

The fix is simple. Change from the classid to the contractid and delete the
static classid.
Assignee: p_ch → cst
Attached patch xpfe patch — — Splinter Review 

        Attachment #171002 -
        Flags: superreview?(neil.parkwaycc.co.uk)

        Attachment #171002 -
        Flags: review?(timeless)

    
    

    
  


  
I don't know if this one needs superreview.

        Attachment #171003 -
        Flags: review?(mconnor)
Status: NEW → ASSIGNED
Whiteboard: [cst:active,r?]
Target Milestone: --- → mozilla1.8beta

    
  

        Attachment #171002 -
        Flags: review?(timeless) → review+

    
    

    
  
*** Bug 258380 has been marked as a duplicate of this bug. ***

    
    

    
  
Comment on attachment 171002 [details] [diff] [review]
xpfe patch

sr=me but not for the reason in the bug description.

        Attachment #171002 -
        Flags: superreview?(neil.parkwaycc.co.uk) → superreview+

        Attachment #171003 -
        Flags: review?(mconnor) → review+
Whiteboard: [cst:active,r?] → checkin

    
    

    
  
checked in
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Whiteboard: checkin

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: