Closed Bug 1186263 Opened 6 years ago Closed 6 years ago
Home panel memory leak
testAboutHomeDisposed is failing, meaning we have a memory leak somewhere.
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: https://github.com/mozilla/firefox-ios/commit/42dab0cc75658fb4dd4a7da365dab7cbee42f3e9 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.
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?
I fixed the failing ViewMemoryLeakTests when working on bug 1178320 (see  & ). 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 ). Stephan, maybe you could give my pull request another review (or close it in favor of your branch).  https://bugzilla.mozilla.org/show_bug.cgi?id=1178320  https://github.com/mozilla/firefox-ios/pull/1229  https://github.com/kif-framework/KIF/issues/739
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.
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 1178320
You need to log in before you can comment on or make changes to this bug.