Fix crash reporting and telemetry switches always show off. Fixes #844

MozillaReality · Nov 27, 2018 · dcb38f3 · dcb38f3
1 parent f5e522d
commit dcb38f3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/views/HoneycombSwitch.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/views/HoneycombSwitch.java
@@ -55,6 +55,10 @@ private void initialize(Context aContext) {
         mStateText = findViewById(R.id.honeycombSwitchStateText);
     }
 
+    public void setChecked(boolean aChecked) {
+        mSwitch.setChecked(aChecked);
+    }
+
     private CompoundButton.OnCheckedChangeListener mInternalSwitchListener = new CompoundButton.OnCheckedChangeListener() {
         @Override
         public void onCheckedChanged(CompoundButton compoundButton, boolean b) {

diff --git a/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/dialogs/SettingsWidget.java b/app/src/common/shared/org/mozilla/vrbrowser/ui/widgets/dialogs/SettingsWidget.java
@@ -120,6 +120,7 @@ private void initialize(Context aContext) {
         });
 
         HoneycombSwitch crashSwitch = findViewById(R.id.crashReportingSwitch);
+        crashSwitch.setChecked(SettingsStore.getInstance(getContext()).isCrashReportingEnabled());
         crashSwitch.setOnCheckedChangeListener((compoundButton, b) -> {
             if (mAudio != null) {
                 mAudio.playSound(AudioEngine.Sound.CLICK);
@@ -129,6 +130,7 @@ private void initialize(Context aContext) {
         });
 
         HoneycombSwitch telemetrySwitch = findViewById(R.id.telemetry_switch);
+        telemetrySwitch.setChecked(SettingsStore.getInstance(getContext()).isTelemetryEnabled());
         telemetrySwitch.setOnCheckedChangeListener((compoundButton, b) -> {
             if (mAudio != null) {
                 mAudio.playSound(AudioEngine.Sound.CLICK);