Fix bug #15109661 Settings crash after tap on back key from print settings

- use onResume() / onPause() for adding / removing the SwitchChangeListener

Change-Id: Ia3a4ff57075d2668e446df4f4f09e4593cc34761
This commit is contained in:
Fabrice Di Meglio
2014-05-20 12:26:16 -07:00
parent e9f007e8df
commit 38f2b27c0d

View File

@@ -143,12 +143,16 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment
mSettingsContentObserver.register(getContentResolver()); mSettingsContentObserver.register(getContentResolver());
updateEmptyView(); updateEmptyView();
updateUiForServiceState(); updateUiForServiceState();
mSwitchBar.addOnSwitchChangeListener(this);
mSwitchBar.show();
} }
@Override @Override
public void onPause() { public void onPause() {
mSettingsContentObserver.unregister(getContentResolver()); mSettingsContentObserver.unregister(getContentResolver());
super.onPause(); super.onPause();
mSwitchBar.removeOnSwitchChangeListener(this);
mSwitchBar.hide();
} }
@Override @Override
@@ -158,20 +162,11 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment
updateUiForArguments(); updateUiForArguments();
} }
@Override
public void onStart() {
super.onStart();
mSwitchBar.addOnSwitchChangeListener(this);
mSwitchBar.show();
}
@Override @Override
public void onDestroyView() { public void onDestroyView() {
if (mOldActivityTitle != null) { if (mOldActivityTitle != null) {
getActivity().getActionBar().setTitle(mOldActivityTitle); getActivity().getActionBar().setTitle(mOldActivityTitle);
} }
mSwitchBar.removeOnSwitchChangeListener(this);
mSwitchBar.hide();
super.onDestroyView(); super.onDestroyView();
} }