Skip to content
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

org.apache.cordova.engine.SystemWebView.mContext leak ! #529

Closed
winnerliu opened this issue Oct 26, 2018 · 8 comments
Closed

org.apache.cordova.engine.SystemWebView.mContext leak ! #529

winnerliu opened this issue Oct 26, 2018 · 8 comments
Labels
info-needed / awaiting response Further information is requested support

Comments

@winnerliu
Copy link

SystemWebView memory leak

@janpio
Copy link
Member

janpio commented Oct 30, 2018

Can you please provide some more information?

@janpio janpio closed this as completed Oct 30, 2018
@winnerliu
Copy link
Author

use leakcanary found

@winnerliu
Copy link
Author

@janpio

@brodybits brodybits added info-needed / awaiting response Further information is requested support labels Nov 11, 2018
@brodybits
Copy link
Contributor

Reopening for now. @winnerliu a better explanation to walk us through the issue would be really appreciated since not all of us are familiar with Leakcanary.

@brodybits brodybits reopened this Nov 11, 2018
@pengbs
Copy link

pengbs commented Apr 28, 2019

same issue

@eugenepoplavsky
Copy link

I think I have the similar(or the same) issue.
You can reproduce it after implementing LeakCanary(link) into your "test" project(link).
Screenshot_2019-11-05-23-57-38-980_org apache cordova unittests

Also, you can enable StrictMode(link) and check TestActivity(for example) on InstanceCountViolation

  • I have this error in my app and also in your "test" project. To reproduce - reenter in the activity a few times.
    ```
    StrictMode.setVmPolicy(new StrictMode.VmPolicy.Builder()
    .detectLeakedSqlLiteObjects()
    .detectLeakedClosableObjects()
    .setClassInstanceLimit(TestActivity.class, 1)
    .penaltyLog()
    // .penaltyDeath()
    .build());

@breautek
Copy link
Contributor

I'm unable to reproduce this (but could be using LeakCanary wrong...), but following their get started link, the logcat only produces org.apache.cordova.unittests D/LeakCanary: Installing AppWatcher, and nothing else. I also tried the StrictMode, but that didn't produce any output either.

Can someone confirm if this is still an issue on the current master?

@breautek
Copy link
Contributor

Closing due to staleness.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info-needed / awaiting response Further information is requested support
Projects
None yet
Development

No branches or pull requests

6 participants