Opened on 05/03/2016 at 08:08:36 AM
Last modified on 09/05/2018 at 11:40:04 AM
#4012 new defect
ABB iOS still sometimes crashes due to memory pressure
| Reported by: | mario | Assignee: | ashephard |
|---|---|---|---|
| Priority: | P3 | Milestone: | |
| Module: | Adblock-Browser-for-iOS | Keywords: | cantfix |
| Cc: | Blocked By: | ||
| Blocking: | Platform: | Adblock Browser for iOS | |
| Ready: | yes | Confidential: | no |
| Tester: | Unknown | Verified working: | no |
| Review URL(s): | |||
Description (last modified by mario)
This is a follow-up ticket to #4010 where we fixed a memory leak which caused crashes due to memory pressure. Since closing #4010 the memory usage while browsing websites has greatly improved. However there's still a noticeable difference compared to other apps using UIWebView:
There is no hard and fast amount of content that needs to be loaded, or in a particular way before it crashes. But, it does happen sometimes.
Jetsam is attached.
This was also tested against Opera iOS with no reproduction.
Environment
ABB 1.3.2 (926)
iPhone 6 Plus
iOS 9.3.1
How to reproduce
- Visit content heavy sites such as imgur.com or foxnews.com
- Browse through multiple images and or articles
Observed behaviour
After loading a series of images or articles, the browser crashes.
Expected behaviour
The browser should not crash.
Attachments (2)
Change History (7)
Changed on 05/03/2016 at 08:10:16 AM by mario
Changed on 05/04/2016 at 08:15:24 AM by scheer
comment:2 Changed on 06/06/2016 at 07:54:55 AM by pavelz
comment:3 Changed on 11/03/2016 at 08:46:49 AM by pavelz
- Keywords cantfix added
comment:4 Changed on 09/05/2018 at 11:39:52 AM by ashephard
Posting my findings regarding memory pressure crashes here: https://gitlab.com/eyeo/auxiliary/issue/issues/102
comment:5 Changed on 09/05/2018 at 11:40:04 AM by ashephard
- Owner set to ashephard
Partial memory leak fixes done for 1.4.0 release
chrome.storage calls
https://www.pivotaltracker.com/story/show/120442231
background page deallocation when toggling the extension
https://www.pivotaltracker.com/story/show/120825965