kumi/andOTP
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

#635 - Make sure that we're unregistering lifecycle observers to avoid memory leaks

Browse source
This commit is contained in:
Joshua Soberg 2021-02-03 22:03:23 -05:00
parent 38a5ca656c
commit ba2f911909
1 changed files with 17 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
app/src/main/java/org/shadowice/flocke/andotp/Activities/AuthenticateActivity.java
View file

@ -66,6 +66,7 @@ public class AuthenticateActivity extends BaseActivity
private String newEncryption = "";
private String existingAuthCredentials;
private boolean isAuthUpgrade = false;
private ProcessLifecycleObserver observer;
private TextInputLayout passwordLayout;
private TextInputEditText passwordInput;
@ -109,8 +110,9 @@ public class AuthenticateActivity extends BaseActivity
}
});
observer = new ProcessLifecycleObserver();
ProcessLifecycleOwner.get().getLifecycle()
.addObserver(new ProcessLifecycleObserver());
.addObserver(observer);
getWindow().setSoftInputMode(LayoutParams.SOFT_INPUT_STATE_ALWAYS_VISIBLE);
}
@ -174,6 +176,15 @@ public class AuthenticateActivity extends BaseActivity
setupUiForTaskState(false);
}
private class ProcessLifecycleObserver implements DefaultLifecycleObserver {
@Override
public void onStop(LifecycleOwner owner) {
if (settings.getRelockOnBackground()) {
cancelBackgroundTask();
}
}
}
@Override
public void onResume() {
super.onResume();
@ -201,13 +212,11 @@ public class AuthenticateActivity extends BaseActivity
unlockProgress.setVisibility(isTaskRunning ? View.VISIBLE : View.GONE);
}
private class ProcessLifecycleObserver implements DefaultLifecycleObserver {
@Override
public void onStop(LifecycleOwner owner) {
if (settings.getRelockOnBackground()) {
cancelBackgroundTask();
}
}
@Override
protected void onDestroy() {
super.onDestroy();
ProcessLifecycleOwner.get().getLifecycle()
.removeObserver(observer);
}
@Override