Merge "Disable hidden network spinner on existing networks" into pi-dev
am: ac951a69aa
Change-Id: I2fcd1b9dbd807dee783ea0027ab7c9b8e2ff369e
This commit is contained in:
@@ -220,6 +220,7 @@ public class WifiConfigController implements TextWatcher,
|
||||
mMeteredSettingsSpinner = mView.findViewById(R.id.metered_settings);
|
||||
mHiddenSettingsSpinner = mView.findViewById(R.id.hidden_settings);
|
||||
mHiddenSettingsSpinner.setOnItemSelectedListener(this);
|
||||
mHiddenSettingsSpinner.setEnabled(false);
|
||||
mHiddenWarningView = mView.findViewById(R.id.hidden_settings_warning);
|
||||
mHiddenWarningView.setVisibility(
|
||||
mHiddenSettingsSpinner.getSelectedItemPosition() == NOT_HIDDEN_NETWORK
|
||||
@@ -238,6 +239,8 @@ public class WifiConfigController implements TextWatcher,
|
||||
showIpConfigFields();
|
||||
showProxyFields();
|
||||
mView.findViewById(R.id.wifi_advanced_toggle).setVisibility(View.VISIBLE);
|
||||
// Hidden option can be changed only when the user adds a network manually.
|
||||
mHiddenSettingsSpinner.setEnabled(true);
|
||||
((CheckBox) mView.findViewById(R.id.wifi_advanced_togglebox))
|
||||
.setOnCheckedChangeListener(this);
|
||||
|
||||
|
@@ -258,6 +258,16 @@ public class WifiConfigControllerTest {
|
||||
assertThat(warningView.getVisibility()).isEqualTo(View.GONE);
|
||||
}
|
||||
|
||||
@Test
|
||||
public void hiddenView_isDisabledWhenAppropriate() {
|
||||
View hiddenSpinner = mView.findViewById(R.id.hidden_settings);
|
||||
assertThat(hiddenSpinner.isEnabled()).isFalse();
|
||||
|
||||
mController = new TestWifiConfigController(mConfigUiBase, mView, null /* accessPoint */,
|
||||
WifiConfigUiBase.MODE_CONNECT);
|
||||
assertThat(hiddenSpinner.isEnabled()).isTrue();
|
||||
}
|
||||
|
||||
public class TestWifiConfigController extends WifiConfigController {
|
||||
|
||||
private TestWifiConfigController(
|
||||
|
Reference in New Issue
Block a user