Closed Bug 1186263 Opened 9 years ago Closed 9 years ago

Home panel memory leak


(Firefox for iOS :: Build & Test, defect)

Not set





(Reporter: bnicholson, Assigned: sleroux)


testAboutHomeDisposed is failing, meaning we have a memory leak somewhere.
Assignee: nobody → sleroux
First of all, I guess when I switched out the root view controller from BVC to a UINavigationController, it messed up the tests. I added a quick patch for that:

Something strange is going on though - when I run the test, aboutHomeController doesn't get nil'ed out but when I run the app and put a breakpoint in the deinit method for HomePanelViewController, it gets called everytime which indicates that it is indeed getting released. Any idea on why it would not be nil'ing it out in the test case but does get nil'ed out when the app is run? Was wondering if you've seen this before with KIF.
Flags: needinfo?(bnicholson)
Haven't seen this before, unfortunately. I'm pretty sure these tests were passing previously, though...I wonder if we'd be able to use git bisection to figure out which change broke them?
Flags: needinfo?(bnicholson)
tracking-fxios: + → ---
Component: General → Build & Test
Still failing?
I fixed the failing ViewMemoryLeakTests when working on bug 1178320 (see [1] & [2]). Sorry, I didn't know there was a duplicate bug. I was able to work around the missing nilification by using an explicit autorelease pool. It appears to be a bug in KIF since it does not happen when deriving from XCTestCase. I reported a bug upstream (see [3]).

Stephan, maybe you could give my pull request another review (or close it in favor of your branch).

Flags: needinfo?(sleroux)
This is awesome! Thanks for fixing this bug - it was bother me for quite some time. I'll go ahead and mark this bug as a dupe of the other one. I looked over the PR and it looks good.
Closed: 9 years ago
Flags: needinfo?(sleroux)
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.