From 9f38e912fffb573e30c65da3846db663f737fe8f Mon Sep 17 00:00:00 2001
From: Matthew Finkel <Matthew.Finkel@gmail.com>
Date: Mon, 9 Apr 2018 22:36:53 +0000
Subject: [PATCH] Bug 25741 - TBA: Disable QR Code reader by default

This is another accessibility and usability concern, so we allow
this as an opt-in.
---
 .../android/app/src/main/res/xml/preferences_accessibility.xml  | 2 +-
 .../base/java/org/mozilla/gecko/toolbar/ToolbarEditLayout.java  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/mobile/android/app/src/main/res/xml/preferences_accessibility.xml b/mobile/android/app/src/main/res/xml/preferences_accessibility.xml
index c82006ee24736..c5afa4d8d6441 100644
--- a/mobile/android/app/src/main/res/xml/preferences_accessibility.xml
+++ b/mobile/android/app/src/main/res/xml/preferences_accessibility.xml
@@ -23,6 +23,6 @@
     <SwitchPreference android:key="android.not_a_preference.qrcode_enabled"
                       android:title="@string/pref_qrcode_enabled"
                       android:summary="@string/pref_qrcode_enabled_summary"
-                      android:defaultValue="true"/>
+                      android:defaultValue="false"/>
 
 </PreferenceScreen>
diff --git a/mobile/android/base/java/org/mozilla/gecko/toolbar/ToolbarEditLayout.java b/mobile/android/base/java/org/mozilla/gecko/toolbar/ToolbarEditLayout.java
index 1d53de172c478..a288472853504 100644
--- a/mobile/android/base/java/org/mozilla/gecko/toolbar/ToolbarEditLayout.java
+++ b/mobile/android/base/java/org/mozilla/gecko/toolbar/ToolbarEditLayout.java
@@ -324,7 +324,7 @@ public class ToolbarEditLayout extends ThemedLinearLayout {
             return false;
         }
         return GeckoSharedPrefs.forApp(context)
-                .getBoolean(GeckoPreferences.PREFS_QRCODE_ENABLED, true);
+                .getBoolean(GeckoPreferences.PREFS_QRCODE_ENABLED, false);
     }
 
     private void launchQRCodeReader() {
-- 
GitLab