joy/android_packages_modules_Connectivity
Watch
1
Fork
You've already forked android_packages_modules_Connectivity
 0
Code Issues Pull requests Projects Releases Wiki Activity

Avoid unregistering the same receiver more than once.

Browse source 
Bug: 231496005
Test: atest com.android.cts.net.HostsideRestrictBackgroundNetworkTests
Change-Id: Iba17daa70f14bdee1d1a7d32cb9dc64623ba1d24
Merged-In: Iba17daa70f14bdee1d1a7d32cb9dc64623ba1d24
This commit is contained in:
Sudheer Shanka 2022-05-04 18:17:38 -07:00
commit d1e236eb05
2 changed files with 20 additions and 10 deletions
Download patch file Download diff file Expand all files Collapse all files

1
tests/cts/hostside/app2/Android.bp
View file

@ -23,6 +23,7 @@ android_test_helper_app {
defaults: ["cts_support_defaults"],
sdk_version: "test_current",
static_libs: [
"androidx.annotation_annotation",
"CtsHostsideNetworkTestsAidl",
"NetworkStackApiStableShims",
],

29
tests/cts/hostside/app2/src/com/android/cts/net/hostside/app2/MyActivity.java
View file

@ -30,6 +30,8 @@ import android.os.Bundle;
import android.os.RemoteException;
import android.util.Log;
import androidx.annotation.GuardedBy;
import com.android.cts.net.hostside.INetworkStateObserver;
/**
@ -37,6 +39,7 @@ import com.android.cts.net.hostside.INetworkStateObserver;
*/
public class MyActivity extends Activity {
@GuardedBy("this")
private BroadcastReceiver finishCommandReceiver = null;
@Override
@ -47,8 +50,11 @@ public class MyActivity extends Activity {
@Override
public void finish() {
if (finishCommandReceiver != null) {
unregisterReceiver(finishCommandReceiver);
synchronized (this) {
if (finishCommandReceiver != null) {
unregisterReceiver(finishCommandReceiver);
finishCommandReceiver = null;
}
}
super.finish();
}
@ -71,14 +77,17 @@ public class MyActivity extends Activity {
super.onResume();
Log.d(TAG, "MyActivity.onResume(): " + getIntent());
Common.notifyNetworkStateObserver(this, getIntent(), TYPE_COMPONENT_ACTIVTY);
finishCommandReceiver = new BroadcastReceiver() {
@Override
public void onReceive(Context context, Intent intent) {
Log.d(TAG, "Finishing MyActivity");
MyActivity.this.finish();
}
};
registerReceiver(finishCommandReceiver, new IntentFilter(ACTION_FINISH_ACTIVITY));
synchronized (this) {
finishCommandReceiver = new BroadcastReceiver() {
@Override
public void onReceive(Context context, Intent intent) {
Log.d(TAG, "Finishing MyActivity");
MyActivity.this.finish();
}
};
registerReceiver(finishCommandReceiver, new IntentFilter(ACTION_FINISH_ACTIVITY),
Context.RECEIVER_EXPORTED);
}
}
@Override