Bug 1401982 - adding restart confirmation prompt when accessibility.force_disable is toggled. r=jaws
authorYura Zenevich <yura.zenevich@gmail.com>
Thu, 21 Sep 2017 11:42:27 -0400
changeset 433104 6c68def04ec4490b4f8d055d350533f37600d7b2
parent 433103 14298b92236cfcfdd0b9dc43d8077e576f775b4d
child 433105 6ac9cbe03c9d1ad4317308f79734b9dd88886f92
push id8114
push userjlorenzo@mozilla.com
push dateThu, 02 Nov 2017 16:33:21 +0000
treeherdermozilla-beta@73e0d89a540f [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjaws
bugs1401982
milestone58.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1401982 - adding restart confirmation prompt when accessibility.force_disable is toggled. r=jaws MozReview-Commit-ID: 4VIu3Dl7zlr
browser/components/preferences/in-content/privacy.js
--- a/browser/components/preferences/in-content/privacy.js
+++ b/browser/components/preferences/in-content/privacy.js
@@ -1642,11 +1642,18 @@ var gPrivacyPane = {
   _initA11yString() {
     let a11yLearnMoreLink =
       Services.urlFormatter.formatURLPref("accessibility.support.url");
     document.getElementById("a11yLearnMoreLink")
       .setAttribute("href", a11yLearnMoreLink);
   },
 
   updateA11yPrefs(checked) {
-    Services.prefs.setIntPref("accessibility.force_disabled", checked ? 1 : 0);
+    let buttonIndex = confirmRestartPrompt(checked, 0, true, false);
+    if (buttonIndex == CONFIRM_RESTART_PROMPT_RESTART_NOW) {
+      Services.prefs.setIntPref("accessibility.force_disabled", checked ? 1 : 0);
+      Services.startup.quit(Ci.nsIAppStartup.eAttemptQuit | Ci.nsIAppStartup.eRestart);
+    }
+
+    // Revert the checkbox in case we didn't quit
+    document.getElementById("a11yPrivacyCheckbox").checked = !checked;
   }
 };