Fix a monkey crash in Developer settings.

- DevSetting shouldn't add switchbar listener if it's a Monkey test run
- DevSetting shouldn't remove switchbar listener if it's not registered
- Refactor some switchbar logic in order to create test

Change-Id: Ie65ff9b04778075789010169039393bd0551acfb
Fix: 64021498
Test: robotests
This commit is contained in:
Fan Zhang
2017-07-25 10:30:24 -07:00
parent a39daa25e7
commit 84dd783c36
4 changed files with 163 additions and 18 deletions

View File

@@ -354,6 +354,7 @@ public class DevelopmentSettings extends RestrictedSettingsFragment
private Dialog mLogpersistClearDialog;
private DashboardFeatureProvider mDashboardFeatureProvider;
private DevelopmentSettingsEnabler mSettingsEnabler;
private DevelopmentSwitchBarController mSwitchBarController;
private BugReportPreferenceController mBugReportController;
private BugReportInPowerPreferenceController mBugReportInPowerController;
private TelephonyMonitorPreferenceController mTelephonyMonitorController;
@@ -622,18 +623,9 @@ public class DevelopmentSettings extends RestrictedSettingsFragment
public void onActivityCreated(Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState);
final SettingsActivity activity = (SettingsActivity) getActivity();
mSwitchBar = activity.getSwitchBar();
if (mUnavailable) {
mSwitchBar.setEnabled(false);
return;
}
if (Utils.isMonkeyRunning()) {
return;
}
mSwitchBar.addOnSwitchChangeListener(this);
mSwitchBar = ((SettingsActivity) getActivity()).getSwitchBar();
mSwitchBarController = new DevelopmentSwitchBarController(
this /* DevelopmentSettings */, mSwitchBar, !mUnavailable, getLifecycle());
}
private boolean removePreferenceForProduction(Preference preference) {
@@ -760,8 +752,6 @@ public class DevelopmentSettings extends RestrictedSettingsFragment
if (mUnavailable) {
return;
}
mSwitchBar.removeOnSwitchChangeListener(this);
mSwitchBar.hide();
getActivity().unregisterReceiver(mUsbReceiver);
getActivity().unregisterReceiver(mBluetoothA2dpReceiver);
BluetoothAdapter adapter = BluetoothAdapter.getDefaultAdapter();