Closed Bug 659580 Opened 12 years ago Closed 12 years ago

Flush CanvasImageCache from xpcom shutdown

Categories

(Core :: DOM: Core & HTML, defect)

Product:
Core
Component:
DOM: Core & HTML
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla7

People

(Reporter: peterv, Assigned: peterv)

References

Details

Attachments

(1 file)

v1
3.94 KB, patch
roc
: review+
Details | Diff | Splinter Review 
This doesn't fix any real leaks, but it makes debugging slightly easier. The problem is that if you quit the browser before all cache items have expired we leak the cache (cache holds elements and is deleted from nsLayoutStatics::Shutdown which gets called after all elements have gone away).
Attached patch v1 — — Splinter Review 

    
  

        Attachment #535005 -
        Flags: review?(roc)

    
    

    
  
Comment on attachment 535005 [details] [diff] [review]
v1

Review of attachment 535005 [details] [diff] [review]:
-----------------------------------------------------------------

        Attachment #535005 -
        Flags: review?(roc) → review+

    
    

    
  
http://hg.mozilla.org/mozilla-central/rev/5229dcb70989
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla7
Component: DOM → DOM: Core & HTML

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