[tor-commits] [sandboxed-tor-browser/master] Don't mess with the security slider at all.

[yawning at torproject.org]

commit d2944eae3c903e7bffec2c6e3249a8dd4a432c8b
Author: Yawning Angel <yawning at schwanenlied.me>
Date:   Fri Dec 23 03:30:22 2016 +0000

    Don't mess with the security slider at all.
    
    GeKo seems to think it will drive users away, and more to the point,
    the fucking `defaultPref` directive doesn't appear to actually take,
    while `pref` and `lockPref` with the same arguments does.
---
 ChangeLog                  | 1 -
 data/installer/mozilla.cfg | 5 +++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index ae90e9c..bdd0e2e 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -10,7 +10,6 @@ Changes in version 0.0.3 - UNRELEASED:
  * Bug 20806: Try even harder to exclude gstreamer.
  * Add a `-h` command line flag that prints usage.
  * Remove the unused `pulseAudioBox` identifier.
- * Set the security slider to "High" by default on a fresh install.
  * Store the version of the sandbox in the config file, and re-Sync() the
    config, and reinstall the `mozilla.cfg` when things change.
  * Include the git revision as a static asset, and display it as part of
diff --git a/data/installer/mozilla.cfg b/data/installer/mozilla.cfg
index 4322906..d4448d0 100644
--- a/data/installer/mozilla.cfg
+++ b/data/installer/mozilla.cfg
@@ -17,5 +17,6 @@ lockPref("extensions.torbutton.versioncheck_enabled", false);
 lockPref("extensions.update.enabled", false);
 lockPref("extensions.update.autoUpdateDefault", false);
 
-// Set the security slider to "High" by default.
-defaultPref("extensions.torbutton.security_slider", 1);
+// Set the security slider to "High" by default, if the fucking `defaultPref`
+// directive worked as advertised.  But it doesn't appear to actually take.
+// defaultPref("extensions.torbutton.security_slider", 1);