Bug 1384962 - Don't set process count for user on Preferences page load, r=jaws
This is a follow-up to incomplete fix in
bug 1382649. In that bug, I attempted to
set the process count value to the e10s rollout value, when the user unchecks
"Use default performance settings". The same function updatePerformanceSettingsBox()
is actually run during page load, resulting the user set value being changed to
e10s rollout value if the user decided to use the default process count.
Tests are written for both patches.
MozReview-Commit-ID: 4CLLcEKbS58
--- a/browser/components/preferences/in-content-new/main.js
+++ b/browser/components/preferences/in-content-new/main.js
@@ -187,19 +187,19 @@ var gMainPane = {
let performanceSettingsUrl = Services.urlFormatter.formatURLPref("app.support.baseURL") + "performance";
performanceSettingsLink.setAttribute("href", performanceSettingsUrl);
this.updateDefaultPerformanceSettingsPref();
let defaultPerformancePref =
document.getElementById("browser.preferences.defaultPerformanceSettings.enabled");
defaultPerformancePref.addEventListener("change", () => {
- this.updatePerformanceSettingsBox();
+ this.updatePerformanceSettingsBox({duringChangeEvent: true});
});
- this.updatePerformanceSettingsBox();
+ this.updatePerformanceSettingsBox({duringChangeEvent: false});
// set up the "use current page" label-changing listener
this._updateUseCurrentButton();
window.addEventListener("focus", this._updateUseCurrentButton.bind(this));
this.updateBrowserStartupLastSession();
if (AppConstants.platform == "win") {
@@ -1109,33 +1109,34 @@ var gMainPane = {
let processCountPref = document.getElementById("dom.ipc.processCount");
let accelerationPref = document.getElementById("layers.acceleration.disabled");
if (processCountPref.value != processCountPref.defaultValue ||
accelerationPref.value != accelerationPref.defaultValue) {
defaultPerformancePref.value = false;
}
},
- updatePerformanceSettingsBox() {
+ updatePerformanceSettingsBox({duringChangeEvent}) {
let defaultPerformancePref =
document.getElementById("browser.preferences.defaultPerformanceSettings.enabled");
let performanceSettings = document.getElementById("performanceSettings");
let processCountPref = document.getElementById("dom.ipc.processCount");
if (defaultPerformancePref.value) {
let accelerationPref = document.getElementById("layers.acceleration.disabled");
// Unset the value so process count will be decided by e10s rollout.
processCountPref.value = processCountPref.defaultValue;
accelerationPref.value = accelerationPref.defaultValue;
performanceSettings.hidden = true;
} else {
let e10sRolloutProcessCountPref =
document.getElementById("dom.ipc.processCount.web");
// Take the e10s rollout value as the default value (if it exists),
// but don't overwrite the user set value.
- if (e10sRolloutProcessCountPref.value &&
+ if (duringChangeEvent &&
+ e10sRolloutProcessCountPref.value &&
processCountPref.value == processCountPref.defaultValue) {
processCountPref.value = e10sRolloutProcessCountPref.value;
}
performanceSettings.hidden = false;
}
},
buildContentProcessCountMenuList() {
--- a/browser/components/preferences/in-content-new/tests/browser.ini
+++ b/browser/components/preferences/in-content-new/tests/browser.ini
@@ -39,16 +39,18 @@ skip-if = os != "win" || (os == "win" &&
skip-if = true || !healthreport # Bug 1185403 for the "true"
[browser_homepages_filter_aboutpreferences.js]
[browser_layersacceleration.js]
[browser_masterpassword.js]
[browser_notifications_do_not_disturb.js]
[browser_password_management.js]
[browser_performance.js]
skip-if = !e10s
+[browser_performance_e10srollout.js]
+skip-if = !e10s
[browser_performance_non_e10s.js]
skip-if = e10s
[browser_permissions_urlFieldHidden.js]
[browser_proxy_backup.js]
[browser_privacypane_1.js]
[browser_privacypane_3.js]
[browser_privacypane_4.js]
[browser_privacypane_5.js]
new file mode 100644
--- /dev/null
+++ b/browser/components/preferences/in-content-new/tests/browser_performance_e10srollout.js
@@ -0,0 +1,102 @@
+const DEFAULT_HW_ACCEL_PREF = Services.prefs.getDefaultBranch(null).getBoolPref("layers.acceleration.disabled");
+const DEFAULT_PROCESS_COUNT = Services.prefs.getDefaultBranch(null).getIntPref("dom.ipc.processCount");
+
+add_task(async function() {
+ await SpecialPowers.pushPrefEnv({set: [
+ ["layers.acceleration.disabled", DEFAULT_HW_ACCEL_PREF],
+ ["dom.ipc.processCount", DEFAULT_PROCESS_COUNT],
+ ["browser.preferences.defaultPerformanceSettings.enabled", true],
+ ]});
+});
+
+add_task(async function() {
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.contentDocument;
+ let useRecommendedPerformanceSettings = doc.querySelector("#useRecommendedPerformanceSettings");
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), true,
+ "pref value should be true before clicking on checkbox");
+ ok(useRecommendedPerformanceSettings.checked, "checkbox should be checked before clicking on checkbox");
+
+ useRecommendedPerformanceSettings.click();
+
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), false,
+ "pref value should be false after clicking on checkbox");
+ ok(!useRecommendedPerformanceSettings.checked, "checkbox should not be checked after clicking on checkbox");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT + 1, "e10s rollout value should be default value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT + 1, "selected item should be the default one");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});
+
+add_task(async function() {
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", false);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.contentDocument;
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT, "default value should be the current value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT, "selected item should be the default one");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});
+
+add_task(async function() {
+ Services.prefs.setIntPref("dom.ipc.processCount", DEFAULT_PROCESS_COUNT + 2);
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", false);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.contentDocument;
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT + 2, "process count should be the set value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT + 2, "selected item should be the set one");
+
+ let useRecommendedPerformanceSettings = doc.querySelector("#useRecommendedPerformanceSettings");
+ useRecommendedPerformanceSettings.click();
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), true,
+ "pref value should be true after clicking on checkbox");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT,
+ "process count should be default value");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});
--- a/browser/components/preferences/in-content/main.js
+++ b/browser/components/preferences/in-content/main.js
@@ -45,19 +45,19 @@ var gMainPane = {
}
this.buildContentProcessCountMenuList();
this.updateDefaultPerformanceSettingsPref();
let defaultPerformancePref =
document.getElementById("browser.preferences.defaultPerformanceSettings.enabled");
defaultPerformancePref.addEventListener("change", () => {
- this.updatePerformanceSettingsBox();
+ this.updatePerformanceSettingsBox({duringChangeEvent: true});
});
- this.updatePerformanceSettingsBox();
+ this.updatePerformanceSettingsBox({duringChangeEvent: false});
let performanceSettingsLink = document.getElementById("performanceSettingsLearnMore");
let performanceSettingsUrl = Services.urlFormatter.formatURLPref("app.support.baseURL") + "performance";
performanceSettingsLink.setAttribute("href", performanceSettingsUrl);
// set up the "use current page" label-changing listener
this._updateUseCurrentButton();
window.addEventListener("focus", this._updateUseCurrentButton.bind(this));
@@ -420,33 +420,34 @@ var gMainPane = {
let processCountPref = document.getElementById("dom.ipc.processCount");
let accelerationPref = document.getElementById("layers.acceleration.disabled");
if (processCountPref.value != processCountPref.defaultValue ||
accelerationPref.value != accelerationPref.defaultValue) {
defaultPerformancePref.value = false;
}
},
- updatePerformanceSettingsBox() {
+ updatePerformanceSettingsBox({duringChangeEvent}) {
let defaultPerformancePref =
document.getElementById("browser.preferences.defaultPerformanceSettings.enabled");
let performanceSettings = document.getElementById("performanceSettings");
let processCountPref = document.getElementById("dom.ipc.processCount");
if (defaultPerformancePref.value) {
let accelerationPref = document.getElementById("layers.acceleration.disabled");
// Unset the value so process count will be decided by e10s rollout.
processCountPref.value = processCountPref.defaultValue;
accelerationPref.value = accelerationPref.defaultValue;
performanceSettings.hidden = true;
} else {
let e10sRolloutProcessCountPref =
document.getElementById("dom.ipc.processCount.web");
// Take the e10s rollout value as the default value (if it exists),
// but don't overwrite the user set value.
- if (e10sRolloutProcessCountPref.value &&
+ if (duringChangeEvent &&
+ e10sRolloutProcessCountPref.value &&
processCountPref.value == processCountPref.defaultValue) {
processCountPref.value = e10sRolloutProcessCountPref.value;
}
performanceSettings.hidden = false;
}
},
buildContentProcessCountMenuList() {
--- a/browser/components/preferences/in-content/tests/browser.ini
+++ b/browser/components/preferences/in-content/tests/browser.ini
@@ -27,16 +27,18 @@ skip-if = os != "win" # This test tests
[browser_defaultbrowser_alwayscheck.js]
[browser_healthreport.js]
skip-if = true || !healthreport # Bug 1185403 for the "true"
[browser_homepages_filter_aboutpreferences.js]
[browser_notifications_do_not_disturb.js]
[browser_password_management.js]
[browser_performance.js]
skip-if = !e10s
+[browser_performance_e10srollout.js]
+skip-if = !e10s
[browser_performance_non_e10s.js]
skip-if = e10s
[browser_permissions_urlFieldHidden.js]
[browser_proxy_backup.js]
[browser_privacypane_1.js]
[browser_privacypane_3.js]
[browser_privacypane_4.js]
[browser_privacypane_5.js]
new file mode 100644
--- /dev/null
+++ b/browser/components/preferences/in-content/tests/browser_performance_e10srollout.js
@@ -0,0 +1,102 @@
+const DEFAULT_HW_ACCEL_PREF = Services.prefs.getDefaultBranch(null).getBoolPref("layers.acceleration.disabled");
+const DEFAULT_PROCESS_COUNT = Services.prefs.getDefaultBranch(null).getIntPref("dom.ipc.processCount");
+
+add_task(async function() {
+ await SpecialPowers.pushPrefEnv({set: [
+ ["layers.acceleration.disabled", DEFAULT_HW_ACCEL_PREF],
+ ["dom.ipc.processCount", DEFAULT_PROCESS_COUNT],
+ ["browser.preferences.defaultPerformanceSettings.enabled", true],
+ ]});
+});
+
+add_task(async function() {
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", null, {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.selectedBrowser.contentDocument;
+ let useRecommendedPerformanceSettings = doc.querySelector("#useRecommendedPerformanceSettings");
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), true,
+ "pref value should be true before clicking on checkbox");
+ ok(useRecommendedPerformanceSettings.checked, "checkbox should be checked before clicking on checkbox");
+
+ useRecommendedPerformanceSettings.click();
+
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), false,
+ "pref value should be false after clicking on checkbox");
+ ok(!useRecommendedPerformanceSettings.checked, "checkbox should not be checked after clicking on checkbox");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT + 1, "e10s rollout value should be default value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT + 1, "selected item should be the default one");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});
+
+add_task(async function() {
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", false);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", null, {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.selectedBrowser.contentDocument;
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT, "default value should be the current value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT, "selected item should be the default one");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});
+
+add_task(async function() {
+ Services.prefs.setIntPref("dom.ipc.processCount", DEFAULT_PROCESS_COUNT + 2);
+ Services.prefs.setIntPref("dom.ipc.processCount.web", DEFAULT_PROCESS_COUNT + 1);
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", false);
+
+ let prefs = await openPreferencesViaOpenPreferencesAPI("paneGeneral", null, {leaveOpen: true});
+ is(prefs.selectedPane, "paneGeneral", "General pane was selected");
+
+ let doc = gBrowser.selectedBrowser.contentDocument;
+ let performanceSettings = doc.querySelector("#performanceSettings");
+ is(performanceSettings.hidden, false, "performance settings section is shown");
+
+ let contentProcessCount = doc.querySelector("#contentProcessCount");
+ is(contentProcessCount.disabled, false, "process count control should be enabled");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT + 2, "process count should be the set value");
+ is(contentProcessCount.selectedItem.value, DEFAULT_PROCESS_COUNT + 2, "selected item should be the set one");
+
+ let useRecommendedPerformanceSettings = doc.querySelector("#useRecommendedPerformanceSettings");
+ useRecommendedPerformanceSettings.click();
+
+ is(Services.prefs.getBoolPref("browser.preferences.defaultPerformanceSettings.enabled"), true,
+ "pref value should be true after clicking on checkbox");
+ is(Services.prefs.getIntPref("dom.ipc.processCount"), DEFAULT_PROCESS_COUNT,
+ "process count should be default value");
+
+ await BrowserTestUtils.removeTab(gBrowser.selectedTab);
+
+ Services.prefs.clearUserPref("dom.ipc.processCount");
+ Services.prefs.clearUserPref("dom.ipc.processCount.web");
+ Services.prefs.setBoolPref("browser.preferences.defaultPerformanceSettings.enabled", true);
+});