Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ecca6508 authored by Eric Biggers's avatar Eric Biggers Committed by Automerger Merge Worker
Browse files

Stop calling LockPatternUtils.setVisiblePasswordEnabled() am: 08e6612a am: a386c786 

Original change: https://android-review.googlesource.com/c/platform/packages/apps/Settings/+/2700639



Change-Id: I0ee235dd800b5bc7ab5e2ad2b4199500b69ad240
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 6cb3c567 a386c786

src/com/android/settings/security/ShowPasswordPreferenceController.java

+0 −9
Original line number Diff line number Diff line
@@ -17,25 +17,17 @@ 
package com.android.settings.security;



import android.content.Context;

import android.os.UserHandle;

import android.provider.Settings;



import com.android.internal.widget.LockPatternUtils;

import com.android.settings.R;

import com.android.settings.core.TogglePreferenceController;

import com.android.settings.overlay.FeatureFactory;



public class ShowPasswordPreferenceController extends TogglePreferenceController {



    private static final String KEY_SHOW_PASSWORD = "show_password";

    private static final int MY_USER_ID = UserHandle.myUserId();

    private final LockPatternUtils mLockPatternUtils;



    public ShowPasswordPreferenceController(Context context) {

        super(context, KEY_SHOW_PASSWORD);

        mLockPatternUtils = FeatureFactory.getFactory(context)

                .getSecurityFeatureProvider()

                .getLockPatternUtils(context);

    }



    @Override
@@ -48,7 +40,6 @@ public class ShowPasswordPreferenceController extends TogglePreferenceController 
    public boolean setChecked(boolean isChecked) {

        Settings.System.putInt(mContext.getContentResolver(), Settings.System.TEXT_SHOW_PASSWORD,

                isChecked ? 1 : 0);

        mLockPatternUtils.setVisiblePasswordEnabled(isChecked, MY_USER_ID);

        return true;

    }

tests/robotests/src/com/android/settings/security/ShowPasswordPreferenceControllerTest.java

+0 −13
Original line number Diff line number Diff line
@@ -18,20 +18,15 @@ package com.android.settings.security; 


import static com.google.common.truth.Truth.assertThat;



import static org.mockito.Mockito.verify;

import static org.mockito.Mockito.when;



import android.content.ContentResolver;

import android.content.Context;

import android.os.UserHandle;

import android.provider.Settings;



import androidx.preference.Preference;

import androidx.preference.PreferenceScreen;



import com.android.internal.widget.LockPatternUtils;

import com.android.settings.testutils.FakeFeatureFactory;



import org.junit.Before;

import org.junit.Ignore;

import org.junit.Test;
@@ -45,12 +40,9 @@ import org.robolectric.annotation.Config; 
@RunWith(RobolectricTestRunner.class)

public class ShowPasswordPreferenceControllerTest {



    @Mock

    private LockPatternUtils mLockPatternUtils;

    @Mock

    private PreferenceScreen mScreen;



    private FakeFeatureFactory mFeatureFactory;

    private Context mContext;

    private ShowPasswordPreferenceController mController;

    private Preference mPreference;
@@ -59,9 +51,6 @@ public class ShowPasswordPreferenceControllerTest { 
    public void setUp() {

        MockitoAnnotations.initMocks(this);

        mContext = RuntimeEnvironment.application;

        mFeatureFactory = FakeFeatureFactory.setupForTest();

        when(mFeatureFactory.securityFeatureProvider.getLockPatternUtils(mContext))

                .thenReturn(mLockPatternUtils);

        mController = new ShowPasswordPreferenceController(mContext);

        mPreference = new Preference(mContext);

        mPreference.setKey(mController.getPreferenceKey());
@@ -100,7 +89,6 @@ public class ShowPasswordPreferenceControllerTest { 
        mController.onPreferenceChange(mPreference, true);



        assertThat(mController.isChecked()).isTrue();

        verify(mLockPatternUtils).setVisiblePasswordEnabled(true, UserHandle.myUserId());

    }



    @Test
@@ -108,6 +96,5 @@ public class ShowPasswordPreferenceControllerTest { 
        mController.onPreferenceChange(mPreference, false);



        assertThat(mController.isChecked()).isFalse();

        verify(mLockPatternUtils).setVisiblePasswordEnabled(false, UserHandle.myUserId());

    }

}