[TIMOB-12987] Android: Webview architecture problems
GitHub Issue | n/a |
---|---|
Type | Bug |
Priority | Low |
Status | Open |
Resolution | Unresolved |
Affected Version/s | n/a |
Fix Version/s | n/a |
Components | Android |
Labels | core, webview |
Reporter | Vishal Duggal |
Assignee | Unknown |
Created | 2013-03-07T00:13:11.000+0000 |
Updated | 2018-02-28T20:04:25.000+0000 |
Description
During the triage of TIMOB-10526 these are the problems I discovered
1. releaseViews is overridden for TIMOB-7808 to get in parity with iOS which I think is a possible memory leak. Also the behavior in TIMOB-7808 is only on Android and iOS and not on MW. So a case can be made that we do not need parity.
2. This ticket also introduced release() which overrides the method in KrollProxy and does not call the parent method which does something completely different.
3. Webviews are tied to the activity lifecycle which is fine if the window is heavyweight but not if the window is lightweight (Look at the code in onDestroy)
No comments