Bug 1447345 - Change enterprise policy that sets homepage to set it as the default rather than the user value for the pref draft
authorKirk Steuber <ksteuber@mozilla.com>
Tue, 20 Mar 2018 10:43:19 -0700
changeset 770068 92b8e8af42a18c3328402d524778d05f08d97afb
parent 769320 4f1014eb5039bdfdd7a39fb7785d102df1994a6f
push id103308
push userksteuber@mozilla.com
push dateTue, 20 Mar 2018 17:43:48 +0000
bugs1447345
milestone61.0a1
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
browser/components/enterprisepolicies/Policies.jsm
browser/components/enterprisepolicies/tests/EnterprisePolicyTesting.jsm
--- 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) {