Rename Enhanced MAC Randomization to Non-Persistent

Enhanced MAC Randomization has been renamed to Non-Persistent MAC
Randomization, so this change should also be reflected in Settings code.

Bug: 162854429
Test: verify developer toggle for non-persistent MAC still worked
Change-Id: Iafdc72e63d9254e32036b491c2458fd5351e36e4
This commit is contained in:
Quang Luong
2021-06-21 11:04:16 -07:00
parent 89b16b3690
commit 88a091789b
4 changed files with 27 additions and 27 deletions

View File

@@ -279,9 +279,9 @@
android:summary="@string/wifi_scan_throttling_summary" /> android:summary="@string/wifi_scan_throttling_summary" />
<SwitchPreference <SwitchPreference
android:key="wifi_enhanced_mac_randomization" android:key="wifi_non_persistent_mac_randomization"
android:title="@string/wifi_enhanced_mac_randomization" android:title="@string/wifi_non_persistent_mac_randomization"
android:summary="@string/wifi_enhanced_mac_randomization_summary" /> android:summary="@string/wifi_non_persistent_mac_randomization_summary" />
<SwitchPreference <SwitchPreference
android:key="mobile_data_always_on" android:key="mobile_data_always_on"

View File

@@ -505,7 +505,7 @@ public class DevelopmentSettingsDashboardFragment extends RestrictedDashboardFra
controllers.add(new WifiDisplayCertificationPreferenceController(context)); controllers.add(new WifiDisplayCertificationPreferenceController(context));
controllers.add(new WifiVerboseLoggingPreferenceController(context)); controllers.add(new WifiVerboseLoggingPreferenceController(context));
controllers.add(new WifiScanThrottlingPreferenceController(context)); controllers.add(new WifiScanThrottlingPreferenceController(context));
controllers.add(new WifiEnhancedMacRandomizationPreferenceController(context)); controllers.add(new WifiNonPersistentMacRandomizationPreferenceController(context));
controllers.add(new MobileDataAlwaysOnPreferenceController(context)); controllers.add(new MobileDataAlwaysOnPreferenceController(context));
controllers.add(new TetheringHardwareAccelPreferenceController(context)); controllers.add(new TetheringHardwareAccelPreferenceController(context));
controllers.add(new BluetoothDeviceNoNamePreferenceController(context)); controllers.add(new BluetoothDeviceNoNamePreferenceController(context));

View File

@@ -26,30 +26,30 @@ import com.android.settings.core.PreferenceControllerMixin;
import com.android.settingslib.development.DeveloperOptionsPreferenceController; import com.android.settingslib.development.DeveloperOptionsPreferenceController;
/** /**
* Developer option controller for enhanced MAC randomization. * Developer option controller for non-persistent MAC randomization.
*/ */
public class WifiEnhancedMacRandomizationPreferenceController public class WifiNonPersistentMacRandomizationPreferenceController
extends DeveloperOptionsPreferenceController extends DeveloperOptionsPreferenceController
implements Preference.OnPreferenceChangeListener, PreferenceControllerMixin { implements Preference.OnPreferenceChangeListener, PreferenceControllerMixin {
private static final String WIFI_ENHANCED_MAC_RANDOMIZATION_KEY = private static final String WIFI_NON_PERSISTENT_MAC_RANDOMIZATION_KEY =
"wifi_enhanced_mac_randomization"; "wifi_non_persistent_mac_randomization";
private static final String ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG = private static final String NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG =
"enhanced_mac_randomization_force_enabled"; "non_persistent_mac_randomization_force_enabled";
public WifiEnhancedMacRandomizationPreferenceController(Context context) { public WifiNonPersistentMacRandomizationPreferenceController(Context context) {
super(context); super(context);
} }
@Override @Override
public String getPreferenceKey() { public String getPreferenceKey() {
return WIFI_ENHANCED_MAC_RANDOMIZATION_KEY; return WIFI_NON_PERSISTENT_MAC_RANDOMIZATION_KEY;
} }
@Override @Override
public boolean onPreferenceChange(Preference preference, Object newValue) { public boolean onPreferenceChange(Preference preference, Object newValue) {
int isEnabledInt = ((Boolean) newValue) ? 1 : 0; int isEnabledInt = ((Boolean) newValue) ? 1 : 0;
Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.putInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, isEnabledInt); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, isEnabledInt);
return true; return true;
} }
@@ -57,7 +57,7 @@ public class WifiEnhancedMacRandomizationPreferenceController
public void updateState(Preference preference) { public void updateState(Preference preference) {
boolean enabled = false; boolean enabled = false;
if (Settings.Global.getInt(mContext.getContentResolver(), if (Settings.Global.getInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, 0) == 1) { NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, 0) == 1) {
enabled = true; enabled = true;
} }
((SwitchPreference) mPreference).setChecked(enabled); ((SwitchPreference) mPreference).setChecked(enabled);
@@ -67,7 +67,7 @@ public class WifiEnhancedMacRandomizationPreferenceController
protected void onDeveloperOptionsSwitchDisabled() { protected void onDeveloperOptionsSwitchDisabled() {
super.onDeveloperOptionsSwitchDisabled(); super.onDeveloperOptionsSwitchDisabled();
Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.putInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, 0); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, 0);
((SwitchPreference) mPreference).setChecked(false); ((SwitchPreference) mPreference).setChecked(false);
} }
} }

View File

@@ -36,48 +36,48 @@ import org.robolectric.RobolectricTestRunner;
import org.robolectric.RuntimeEnvironment; import org.robolectric.RuntimeEnvironment;
@RunWith(RobolectricTestRunner.class) @RunWith(RobolectricTestRunner.class)
public class WifiEnhancedMacRandomizationPreferenceControllerTest { public class WifiNonPersistentMacRandomizationPreferenceControllerTest {
private static final String ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG = private static final String NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG =
"enhanced_mac_randomization_force_enabled"; "non_persistent_mac_randomization_force_enabled";
@Mock @Mock
private SwitchPreference mPreference; private SwitchPreference mPreference;
@Mock @Mock
private PreferenceScreen mPreferenceScreen; private PreferenceScreen mPreferenceScreen;
private Context mContext; private Context mContext;
private WifiEnhancedMacRandomizationPreferenceController mController; private WifiNonPersistentMacRandomizationPreferenceController mController;
@Before @Before
public void setup() { public void setup() {
MockitoAnnotations.initMocks(this); MockitoAnnotations.initMocks(this);
mContext = RuntimeEnvironment.application; mContext = RuntimeEnvironment.application;
mController = new WifiEnhancedMacRandomizationPreferenceController(mContext); mController = new WifiNonPersistentMacRandomizationPreferenceController(mContext);
when(mPreferenceScreen.findPreference(mController.getPreferenceKey())) when(mPreferenceScreen.findPreference(mController.getPreferenceKey()))
.thenReturn(mPreference); .thenReturn(mPreference);
mController.displayPreference(mPreferenceScreen); mController.displayPreference(mPreferenceScreen);
} }
@Test @Test
public void onPreferenceChanged_enabled_shouldTurnOnEnhancedRandomization() { public void onPreferenceChanged_enabled_shouldTurnOnNonPersistentRandomization() {
mController.onPreferenceChange(mPreference, true /* new value */); mController.onPreferenceChange(mPreference, true /* new value */);
int mode = Settings.Global.getInt(mContext.getContentResolver(), int mode = Settings.Global.getInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, -1); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, -1);
assertThat(mode).isEqualTo(1); assertThat(mode).isEqualTo(1);
} }
@Test @Test
public void onPreferenceChanged_disabled_shouldTurnOffEnhancedRandomization() { public void onPreferenceChanged_disabled_shouldTurnOffNonPersistentRandomization() {
mController.onPreferenceChange(mPreference, false /* new value */); mController.onPreferenceChange(mPreference, false /* new value */);
int mode = Settings.Global.getInt(mContext.getContentResolver(), int mode = Settings.Global.getInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, -1); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, -1);
assertThat(mode).isEqualTo(0); assertThat(mode).isEqualTo(0);
} }
@Test @Test
public void updateState_preferenceShouldBeChecked() { public void updateState_preferenceShouldBeChecked() {
Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.putInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, 1); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, 1);
mController.updateState(mPreference); mController.updateState(mPreference);
verify(mPreference).setChecked(true); verify(mPreference).setChecked(true);
@@ -86,7 +86,7 @@ public class WifiEnhancedMacRandomizationPreferenceControllerTest {
@Test @Test
public void updateState_preferenceShouldNotBeChecked() { public void updateState_preferenceShouldNotBeChecked() {
Settings.Global.putInt(mContext.getContentResolver(), Settings.Global.putInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, 0); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, 0);
mController.updateState(mPreference); mController.updateState(mPreference);
verify(mPreference).setChecked(false); verify(mPreference).setChecked(false);
@@ -97,7 +97,7 @@ public class WifiEnhancedMacRandomizationPreferenceControllerTest {
mController.onDeveloperOptionsSwitchDisabled(); mController.onDeveloperOptionsSwitchDisabled();
int mode = Settings.Global.getInt(mContext.getContentResolver(), int mode = Settings.Global.getInt(mContext.getContentResolver(),
ENHANCED_MAC_RANDOMIZATION_FEATURE_FLAG, -1); NON_PERSISTENT_MAC_RANDOMIZATION_FEATURE_FLAG, -1);
assertThat(mode).isEqualTo(0); assertThat(mode).isEqualTo(0);
assertThat(mPreference.isChecked()).isFalse(); assertThat(mPreference.isChecked()).isFalse();