792511 - Shutdown leaks on TechCrunch

Status: RESOLVED WORKSFORME

(Toolkit :: Storage, defect)

Description

[Andrew McCreight [:mccr8]]

1. Go to a TechCrunch page, like 
  http://techcrunch.com/2012/09/19/take-public-transportation-heres-what-to-expect-from-the-maps-app-on-ios-6/

2. Exit.

Shutdown leak! I've noticed this for a while, I just haven't gotten around to filing it.

Comment 1

[Andrew McCreight [:mccr8]]

Attachment: boat view

Comment 2

[Andrew McCreight [:mccr8]]

This looks very similar to bug 773255.

Comment 3

[Andrew McCreight [:mccr8]]

I've been doing ref count logging, figuring out what is holding something alive, then repeating.

I started with MemoryReporter_StorageSQLite. This is being held alive by moz::storage::Service. This, in turn, appears to be held alive by nsCookieService.

I was unable to get refcount logging to work for that, but the only data structure that seems to hold a pointer to nsICookieService is nsHttpHandler, and there's one of those that is leaking so I assume that's the next link. That could just be a red herring, though.

There is one leaking nsDocShell, maybe I should try examining that. That looks like the most "important" data structure in the bloat log.

Comment 4

[Andrew McCreight [:mccr8]]

This doesn't seem to happen any more.