Commit 2b3f71bc authored by Michael Kaply's avatar Michael Kaply
Browse files

Bug 1571033 - Don't clear user homepage when setting default via policy. r=mconley

Differential Revision: https://phabricator.services.mozilla.com/D40453

--HG--
extra : moz-landing-system : lando
parent a3b25b20
......@@ -877,9 +877,8 @@ var Policies = {
true
);
} else {
runOncePerModification("setHomepage", homepages, () => {
Services.prefs.clearUserPref("browser.startup.homepage");
});
// Clear out old run once modification that is no longer used.
clearRunOnceModification("setHomepage");
}
}
if (param.StartPage) {
......
......@@ -151,19 +151,7 @@ add_task(async function homepage_test_repeat_same_policy_value() {
expectedPageVal: 3,
});
Services.prefs.clearUserPref("browser.startup.page");
});
add_task(async function homepage_test_different_policy_value() {
// This policy is a change from the policy's previous value. This should
// override the user's homepage
await setupPolicyEngineWithJson({
policies: {
Homepage: {
URL: "http://example3.com/",
},
},
});
await check_homepage({ expectedURL: "http://example3.com/" });
Services.prefs.clearUserPref("browser.startup.homepage");
});
add_task(async function homepage_test_empty_additional() {
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment