From cad9420134bbdd97b9c5650ac3b73c26d6f69eab Mon Sep 17 00:00:00 2001 From: Remi NGUYEN VAN Date: Wed, 2 Dec 2020 13:37:51 +0900 Subject: Run LowpanManager on BackgroundThread The ConnectivityThread class is being separated into a specific connectivity JAR; transport-specific managers like LowpanManager should not be sharing the ConnectivityThread. As callbacks from ILowpanManager / ILowpanManagerListener already do not have any ordering guarantee with ConnectivityManager callbacks, the impact of this change should be minimal. LowpanManager is unused in AOSP, and not part of the API. Bug: 174436414 Test: m Change-Id: I23483ed7c4a6c5283b365430a3e503a0dd86c2cb --- lowpan/java/android/net/lowpan/LowpanManager.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'lowpan') diff --git a/lowpan/java/android/net/lowpan/LowpanManager.java b/lowpan/java/android/net/lowpan/LowpanManager.java index 76876ce01c96..33b35e6af7af 100644 --- a/lowpan/java/android/net/lowpan/LowpanManager.java +++ b/lowpan/java/android/net/lowpan/LowpanManager.java @@ -24,6 +24,10 @@ import android.os.IBinder; import android.os.Looper; import android.os.RemoteException; import android.os.ServiceManager; + +import com.android.internal.annotations.VisibleForTesting; +import com.android.internal.os.BackgroundThread; + import java.lang.ref.WeakReference; import java.util.HashMap; import java.util.Map; @@ -97,10 +101,14 @@ public class LowpanManager { * * @param context the application context * @param service the Binder interface - * @param looper the default Looper to run callbacks on * @hide - hide this because it takes in a parameter of type ILowpanManager, which is a system * private class. */ + public LowpanManager(Context context, ILowpanManager service) { + this(context, service, BackgroundThread.get().getLooper()); + } + + @VisibleForTesting public LowpanManager(Context context, ILowpanManager service, Looper looper) { mContext = context; mService = service; -- cgit v1.2.3