Merge "[User Switcher] Remove manage user in phone dialog" into tm-qpr-dev
This commit is contained in:
@@ -26,6 +26,7 @@ import android.view.LayoutInflater;
|
|||||||
import android.view.View;
|
import android.view.View;
|
||||||
import android.view.ViewGroup;
|
import android.view.ViewGroup;
|
||||||
|
|
||||||
|
import androidx.annotation.NonNull;
|
||||||
import androidx.annotation.Nullable;
|
import androidx.annotation.Nullable;
|
||||||
|
|
||||||
import com.android.internal.logging.MetricsLogger;
|
import com.android.internal.logging.MetricsLogger;
|
||||||
@@ -43,6 +44,9 @@ import com.android.systemui.statusbar.policy.BaseUserSwitcherAdapter;
|
|||||||
import com.android.systemui.statusbar.policy.UserSwitcherController;
|
import com.android.systemui.statusbar.policy.UserSwitcherController;
|
||||||
import com.android.systemui.user.data.source.UserRecord;
|
import com.android.systemui.user.data.source.UserRecord;
|
||||||
|
|
||||||
|
import java.util.List;
|
||||||
|
import java.util.stream.Collectors;
|
||||||
|
|
||||||
import javax.inject.Inject;
|
import javax.inject.Inject;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -83,6 +87,13 @@ public class UserDetailView extends PseudoGridView {
|
|||||||
private final FalsingManager mFalsingManager;
|
private final FalsingManager mFalsingManager;
|
||||||
private @Nullable UserSwitchDialogController.DialogShower mDialogShower;
|
private @Nullable UserSwitchDialogController.DialogShower mDialogShower;
|
||||||
|
|
||||||
|
@NonNull
|
||||||
|
@Override
|
||||||
|
protected List<UserRecord> getUsers() {
|
||||||
|
return super.getUsers().stream().filter(
|
||||||
|
userRecord -> !userRecord.isManageUsers).collect(Collectors.toList());
|
||||||
|
}
|
||||||
|
|
||||||
@Inject
|
@Inject
|
||||||
public Adapter(Context context, UserSwitcherController controller,
|
public Adapter(Context context, UserSwitcherController controller,
|
||||||
UiEventLogger uiEventLogger, FalsingManager falsingManager) {
|
UiEventLogger uiEventLogger, FalsingManager falsingManager) {
|
||||||
|
|||||||
@@ -43,11 +43,7 @@ protected constructor(
|
|||||||
}
|
}
|
||||||
|
|
||||||
override fun getCount(): Int {
|
override fun getCount(): Int {
|
||||||
return if (controller.isKeyguardShowing) {
|
return users.size
|
||||||
users.count { !it.isRestricted }
|
|
||||||
} else {
|
|
||||||
users.size
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
||||||
override fun getItem(position: Int): UserRecord {
|
override fun getItem(position: Int): UserRecord {
|
||||||
|
|||||||
@@ -33,6 +33,7 @@ import com.android.systemui.qs.QSUserSwitcherEvent
|
|||||||
import com.android.systemui.statusbar.policy.UserSwitcherController
|
import com.android.systemui.statusbar.policy.UserSwitcherController
|
||||||
import com.android.systemui.user.data.source.UserRecord
|
import com.android.systemui.user.data.source.UserRecord
|
||||||
import org.junit.Assert.assertEquals
|
import org.junit.Assert.assertEquals
|
||||||
|
import org.junit.Assert.assertNull
|
||||||
import org.junit.Before
|
import org.junit.Before
|
||||||
import org.junit.Test
|
import org.junit.Test
|
||||||
import org.junit.runner.RunWith
|
import org.junit.runner.RunWith
|
||||||
@@ -139,6 +140,11 @@ class UserDetailViewAdapterTest : SysuiTestCase() {
|
|||||||
clickableTest(false, false, mUserDetailItemView, true)
|
clickableTest(false, false, mUserDetailItemView, true)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@Test
|
||||||
|
fun testManageUsersIsNotAvailable() {
|
||||||
|
assertNull(adapter.users.find { it.isManageUsers })
|
||||||
|
}
|
||||||
|
|
||||||
private fun createUserRecord(current: Boolean, guest: Boolean) =
|
private fun createUserRecord(current: Boolean, guest: Boolean) =
|
||||||
UserRecord(
|
UserRecord(
|
||||||
UserInfo(0 /* id */, "name", 0 /* flags */),
|
UserInfo(0 /* id */, "name", 0 /* flags */),
|
||||||
|
|||||||
Reference in New Issue
Block a user