status-im / syng-client

The Mobile Client for the Ethereum Network DEPRECATED
Other
11 stars 4 forks source link

Memory Leak with CordovaWebView #13

Closed 0xc1c4da closed 9 years ago

0xc1c4da commented 9 years ago

08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ In io.syng:1.0:1. 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * io.syng.fragment.WebViewFragment has leaked: 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * GC ROOT thread java.lang.Thread.<Java Local, Native Stack> (named 'pool-13-thread-1') 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * references org.apache.cordova.CordovaWebViewImpl$2.this$0 (anonymous class implements java.lang.Runnable) 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * references org.apache.cordova.CordovaWebViewImpl.cordova 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * references io.syng.fragment.WebViewFragment$1.this$0 (anonymous class extends org.apache.cordova.CordovaInterfaceImpl) 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ * leaks io.syng.fragment.WebViewFragment instance 08-10 19:01:13.937 1586-4885/io.syng D/LeakCanary﹕ [ 08-10 19:01:13.937 1586: 4885 D/LeakCanary ]

dmytyar commented 9 years ago

Have no idea what it related to, Removed anonymous class that extend CordovaInterfaceImpl. I not see LeakCanary on my device, so can't check if issue gone. If it left - it's inside Cordova sources.

0xc1c4da commented 9 years ago

wontfix, cordova issue