Merge "[User Switcher] Remove callback in onDestroy" into tm-dev
This commit is contained in:
@@ -71,6 +71,11 @@ class UserSwitcherActivity @Inject constructor(
|
|||||||
private var popupMenu: UserSwitcherPopupMenu? = null
|
private var popupMenu: UserSwitcherPopupMenu? = null
|
||||||
private lateinit var addButton: View
|
private lateinit var addButton: View
|
||||||
private var addUserRecords = mutableListOf<UserRecord>()
|
private var addUserRecords = mutableListOf<UserRecord>()
|
||||||
|
private val userSwitchedCallback: UserTracker.Callback = object : UserTracker.Callback {
|
||||||
|
override fun onUserChanged(newUser: Int, userContext: Context) {
|
||||||
|
finish()
|
||||||
|
}
|
||||||
|
}
|
||||||
// When the add users options become available, insert another option to manage users
|
// When the add users options become available, insert another option to manage users
|
||||||
private val manageUserRecord = UserRecord(
|
private val manageUserRecord = UserRecord(
|
||||||
null /* info */,
|
null /* info */,
|
||||||
@@ -215,11 +220,7 @@ class UserSwitcherActivity @Inject constructor(
|
|||||||
initBroadcastReceiver()
|
initBroadcastReceiver()
|
||||||
|
|
||||||
parent.post { buildUserViews() }
|
parent.post { buildUserViews() }
|
||||||
userTracker.addCallback(object : UserTracker.Callback {
|
userTracker.addCallback(userSwitchedCallback, mainExecutor)
|
||||||
override fun onUserChanged(newUser: Int, userContext: Context) {
|
|
||||||
finish()
|
|
||||||
}
|
|
||||||
}, mainExecutor)
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun showPopupMenu() {
|
private fun showPopupMenu() {
|
||||||
@@ -340,6 +341,7 @@ class UserSwitcherActivity @Inject constructor(
|
|||||||
super.onDestroy()
|
super.onDestroy()
|
||||||
|
|
||||||
broadcastDispatcher.unregisterReceiver(broadcastReceiver)
|
broadcastDispatcher.unregisterReceiver(broadcastReceiver)
|
||||||
|
userTracker.removeCallback(userSwitchedCallback)
|
||||||
}
|
}
|
||||||
|
|
||||||
private fun initBroadcastReceiver() {
|
private fun initBroadcastReceiver() {
|
||||||
|
|||||||
Reference in New Issue
Block a user