Revert "Check for leaking receivers"

This reverts commit c520cb2622.
pull/208/head
M66B 3 years ago
parent a13ffee301
commit be294d35c8

@ -21,11 +21,9 @@ package eu.faircode.email;
import android.Manifest; import android.Manifest;
import android.app.ActivityManager; import android.app.ActivityManager;
import android.content.BroadcastReceiver;
import android.content.ComponentName; import android.content.ComponentName;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.content.IntentFilter;
import android.content.SharedPreferences; import android.content.SharedPreferences;
import android.content.res.Configuration; import android.content.res.Configuration;
import android.graphics.Bitmap; import android.graphics.Bitmap;
@ -79,7 +77,6 @@ abstract class ActivityBase extends AppCompatActivity implements SharedPreferenc
private boolean visible; private boolean visible;
private boolean contacts; private boolean contacts;
private List<IKeyPressedListener> keyPressedListeners = new ArrayList<>(); private List<IKeyPressedListener> keyPressedListeners = new ArrayList<>();
private List<BroadcastReceiver> registeredReceivers = new ArrayList<>();
@Override @Override
protected void attachBaseContext(Context base) { protected void attachBaseContext(Context base) {
@ -350,50 +347,12 @@ abstract class ActivityBase extends AppCompatActivity implements SharedPreferenc
} }
} }
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, int flags) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, flags);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, @Nullable String broadcastPermission, @Nullable Handler scheduler) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, broadcastPermission, scheduler);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, @Nullable String broadcastPermission, @Nullable Handler scheduler, int flags) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, broadcastPermission, scheduler, flags);
}
@Override
public void unregisterReceiver(BroadcastReceiver receiver) {
super.unregisterReceiver(receiver);
registeredReceivers.remove(receiver);
}
@Override @Override
protected void onDestroy() { protected void onDestroy() {
Log.i("Destroy " + this.getClass().getName()); Log.i("Destroy " + this.getClass().getName());
PreferenceManager.getDefaultSharedPreferences(this).unregisterOnSharedPreferenceChangeListener(this); PreferenceManager.getDefaultSharedPreferences(this).unregisterOnSharedPreferenceChangeListener(this);
try { try {
getSupportFragmentManager().unregisterFragmentLifecycleCallbacks(lifecycleCallbacks); getSupportFragmentManager().unregisterFragmentLifecycleCallbacks(lifecycleCallbacks);
Log.i(this.getClass() + " receivers leaking=" + registeredReceivers.size());
for (BroadcastReceiver receiver : registeredReceivers) {
Log.e(this.getClass() + " receiver leaking class=" + receiver.getClass());
unregisterReceiver(receiver);
}
registeredReceivers.clear();
super.onDestroy(); super.onDestroy();
originalContext = null; originalContext = null;
} catch (Throwable ex) { } catch (Throwable ex) {

@ -19,23 +19,16 @@ package eu.faircode.email;
Copyright 2018-2022 by Marcel Bokhorst (M66B) Copyright 2018-2022 by Marcel Bokhorst (M66B)
*/ */
import android.content.BroadcastReceiver;
import android.content.Intent; import android.content.Intent;
import android.content.IntentFilter;
import android.os.Bundle; import android.os.Bundle;
import android.os.Handler; import android.os.Handler;
import androidx.annotation.Nullable;
import androidx.lifecycle.LifecycleService; import androidx.lifecycle.LifecycleService;
import java.util.ArrayList;
import java.util.HashMap; import java.util.HashMap;
import java.util.List;
import java.util.Map; import java.util.Map;
abstract class ServiceBase extends LifecycleService { abstract class ServiceBase extends LifecycleService {
private List<BroadcastReceiver> registeredReceivers = new ArrayList<>();
@Override @Override
public void onCreate() { public void onCreate() {
Map<String, String> crumb = new HashMap<>(); Map<String, String> crumb = new HashMap<>();
@ -62,49 +55,12 @@ abstract class ServiceBase extends LifecycleService {
return super.onStartCommand(intent, flags, startId); return super.onStartCommand(intent, flags, startId);
} }
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, int flags) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, flags);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, @Nullable String broadcastPermission, @Nullable Handler scheduler) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, broadcastPermission, scheduler);
}
@Override
public Intent registerReceiver(@Nullable BroadcastReceiver receiver, IntentFilter filter, @Nullable String broadcastPermission, @Nullable Handler scheduler, int flags) {
registeredReceivers.add(receiver);
return super.registerReceiver(receiver, filter, broadcastPermission, scheduler, flags);
}
@Override
public void unregisterReceiver(BroadcastReceiver receiver) {
super.unregisterReceiver(receiver);
registeredReceivers.remove(receiver);
}
@Override @Override
public void onDestroy() { public void onDestroy() {
Map<String, String> crumb = new HashMap<>(); Map<String, String> crumb = new HashMap<>();
crumb.put("state", "destroy"); crumb.put("state", "destroy");
Log.breadcrumb(this.getClass().getSimpleName(), crumb); Log.breadcrumb(this.getClass().getSimpleName(), crumb);
Log.i(this.getClass() + " receivers leaking=" + registeredReceivers.size());
for (BroadcastReceiver receiver : registeredReceivers) {
Log.e(this.getClass() + " receiver leaking class=" + receiver.getClass());
unregisterReceiver(receiver);
}
registeredReceivers.clear();
super.onDestroy(); super.onDestroy();
} }

Loading…
Cancel
Save