Bug 1447345 - Change enterprise policy that sets homepage to set it as the default rather than the user value for the pref
Additionally removes a stray debugging line that I accidentally checked into the tree previously.
MozReview-Commit-ID: 7F2S8WBgKCj
--- a/browser/components/enterprisepolicies/Policies.jsm
+++ b/browser/components/enterprisepolicies/Policies.jsm
@@ -293,19 +293,21 @@ var Policies = {
}
if (param.Locked) {
setAndLockPref("browser.startup.homepage", homepages);
setAndLockPref("browser.startup.page", 1);
setAndLockPref("pref.browser.homepage.disable_button.current_page", true);
setAndLockPref("pref.browser.homepage.disable_button.bookmark_page", true);
setAndLockPref("pref.browser.homepage.disable_button.restore_default", true);
} else {
+ setDefaultPref("browser.startup.homepage", homepages);
+ setDefaultPref("browser.startup.page", 1);
runOncePerModification("setHomepage", homepages, () => {
- Services.prefs.setStringPref("browser.startup.homepage", homepages);
- Services.prefs.setIntPref("browser.startup.page", 1);
+ Services.prefs.clearUserPref("browser.startup.homepage");
+ Services.prefs.clearUserPref("browser.startup.page");
});
}
}
},
"InstallAddons": {
onBeforeUIStartup(manager, param) {
addAllowDenyPermissions("install", param.Allow, null);
--- a/browser/components/enterprisepolicies/tests/EnterprisePolicyTesting.jsm
+++ b/browser/components/enterprisepolicies/tests/EnterprisePolicyTesting.jsm
@@ -28,17 +28,16 @@ this.EnterprisePolicyTesting = {
filePath = json;
}
Services.prefs.setStringPref("browser.policies.alternatePath", filePath);
let promise = new Promise(resolve => {
Services.obs.addObserver(function observer() {
Services.obs.removeObserver(observer, "EnterprisePolicies:AllPoliciesApplied");
- dump(`bytesized: setupPolicyEngineWithJson resolving`);
resolve();
}, "EnterprisePolicies:AllPoliciesApplied");
});
// Clear any previously used custom schema
Cu.unload("resource:///modules/policies/schema.jsm");
if (customSchema) {