[tbb-commits] [tor-browser/tor-browser-60.6.1esr-8.5-1] fixup! Bug 28329 - Part 4. Add new Tor Bootstrapping and configuration screens

[gk at torproject.org]

commit 937038361e7529779f5bfce42b7e2e4007542503
Author: Georg Koppen <gk at torproject.org>
Date:   Fri May 10 07:54:23 2019 +0000

    fixup! Bug 28329 - Part 4. Add new Tor Bootstrapping and configuration screens
---
 .../mozilla/gecko/torbootstrap/TorPreferences.java | 25 ++++++++++++----------
 1 file changed, 14 insertions(+), 11 deletions(-)

diff --git a/mobile/android/base/java/org/mozilla/gecko/torbootstrap/TorPreferences.java b/mobile/android/base/java/org/mozilla/gecko/torbootstrap/TorPreferences.java
index 87ce1ec4bec6..87b752ab5991 100644
--- a/mobile/android/base/java/org/mozilla/gecko/torbootstrap/TorPreferences.java
+++ b/mobile/android/base/java/org/mozilla/gecko/torbootstrap/TorPreferences.java
@@ -81,12 +81,12 @@ import static org.mozilla.gecko.preferences.GeckoPreferences.NON_PREF_PREFIX;
  *   PREFS_BRIDGES_ENABLED
  *   PREFS_BRIDGES_TYPE
  *   PREFS_BRIDGES_PROVIDE
- *   pref_bridges_enabled (Orbot)
- *   pref_bridges_list    (Orbot)
+ *   pref_bridges_enabled (tor-android-service)
+ *   pref_bridges_list    (tor-android-service)
  *
  * These may be in following three end states where PREFS_BRIDGES_ENABLED and
  * pref_bridges_enabled must always match, and pref_bridges_list must either match
- * PREFS_BRIDGES_PROVIDE or contain a list of bridges of type PREFS_BRIDGES_TYPE.
+ * PREFS_BRIDGES_PROVIDE or contain type PREFS_BRIDGES_TYPE.
  *
  *   PREFS_BRIDGES_ENABLED=false
  *   PREFS_BRIDGES_TYPE=null
@@ -98,7 +98,7 @@ import static org.mozilla.gecko.preferences.GeckoPreferences.NON_PREF_PREFIX;
  *   PREFS_BRIDGES_TYPE=T1
  *   PREFS_BRIDGES_PROVIDE=null
  *   pref_bridges_enabled=true
- *   pref_bridges_list=X1
+ *   pref_bridges_list=T1
  *
  *   PREFS_BRIDGES_ENABLED=true
  *   PREFS_BRIDGES_TYPE=null
@@ -287,7 +287,8 @@ public class TorPreferences extends AppCompatPreferenceActivity {
         // Save the bridgesType with the PREFS_BRIDGES_TYPE pref as the key
         // (for future lookup). If bridgesType is null, then save the
         // bridgesLines with the PREFS_BRIDGES_PROVIDE pref as the key, and
-        // use Orbot's helper method and enable Orbot's bridge pref.
+        // use tor-android-service's helper method and enable
+        // tor-android-service's bridge pref.
         protected boolean setBridges(SharedPreferences.Editor editor, String bridgesType, String bridgesLines) {
             if (editor == null) {
                 Log.w(LOGTAG, "setBridges: editor is null");
@@ -310,12 +311,12 @@ public class TorPreferences extends AppCompatPreferenceActivity {
                 return false;
             }
 
-            // Set Orbot's preference
+            // Set tor-android service's preference
             Prefs.setBridgesList(bridgesLines);
 
             // If either of these are not null, then we're enabling bridges
             boolean bridgesAreEnabled = (bridgesType != null) || (bridgesLines != null);
-            // Inform Orbot bridges are enabled
+            // Inform tor-android-service bridges are enabled
             Prefs.putBridgesEnabled(bridgesAreEnabled);
             return true;
         }
@@ -555,7 +556,7 @@ public class TorPreferences extends AppCompatPreferenceActivity {
 
         // We follow this logic:
         //   If the bridgesEnabled switch is off, then false
-        //   If Orbot doesn't have bridges enabled, then false
+        //   If tor-android-service doesn't have bridges enabled, then false
         //   If PREFS_BRIDGES_PROVIDE is not null, then true
         //   Else false
         private boolean isBridgeProvided(SwitchPreference bridgesEnabled) {
@@ -733,8 +734,9 @@ public class TorPreferences extends AppCompatPreferenceActivity {
             // so replace the string with only the pluggable transport name.
             // This will need updating when another transport is "recommended".
             //
-            // Similarly, if meek-azure is chosen, substitute it with "meek" (Orbot
-            // only handles these keywords specially if they are less than 5 characters).
+            // Similarly, if meek-azure is chosen, substitute it with "meek"
+            // (tor-android-service only handles these keywords specially if
+            // they are less than 5 characters).
             if (bridgesType.contains("obfs4")) {
                 bridgesType = "obfs4";
             } else if (bridgesType.contains("meek-azure")) {
@@ -963,7 +965,8 @@ public class TorPreferences extends AppCompatPreferenceActivity {
                 return;
             }
 
-            // Set the preferences (both our preference and Orbot's preference)
+            // Set the preferences (both our preference and
+            // tor-android-service's preference)
             Log.w(LOGTAG, "Saving Bridge preference: " + bridgesLines);
             if (!setBridges(bridgesProvide.getEditor(), null, bridgesLines)) {
                 // TODO inform the user