New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[TIMOB-24206] iOS: Fix image leaks on JSCore, use JSCore by default inside iOS-core as well #9850
Conversation
Generated by 🚫 dangerJS |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
On iOS 11, it is crashing after selecting 30 pics/approx 3 mints.
This reverts commit 095911e.
@vijaysingh-axway PR updated with the regards to the discussion on Teams. We have to extremely sensitive tests to ensure the JSGarbageCollect plays nice. |
iphone/Classes/TiUIView.m
Outdated
@@ -605,6 +605,7 @@ - (void)setBackgroundImage_:(id)image | |||
if (backgroundRepeat) { | |||
[self renderRepeatedBackground:bgImage]; | |||
} else { | |||
RELEASE_TO_NIL([self backgroundImageLayer].contents); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is causing crash. If I remove this line, JSGarbageCollect is working nice. I'll do more testing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CR passed.
JIRA: https://jira.appcelerator.org/browse/TIMOB-24206