Bug 1411517 - stylo: "layout.css.stylo-blocklist.enabled" pref's default value should be false. draft
authorJeremy Chen <jeremychen@mozilla.com>
Thu, 26 Oct 2017 22:19:08 +0200
changeset 687059 7ecffbdeff3a82b79fa04685a4dbd2950520c2b2
parent 687040 7290c8fce8a1f5c4bce930d951eaf1d6326bd788
child 737566 a17988ae06e64814a9f070ca00d677badf6cb0ea
push id86403
push userbmo:jeremychen@mozilla.com
push dateThu, 26 Oct 2017 20:27:48 +0000
bugs1411517
milestone58.0a1
Bug 1411517 - stylo: "layout.css.stylo-blocklist.enabled" pref's default value should be false. At present, the "layout.css.stylo-blocklist.blocked_domains" pref is empty. It is probably meaningless to set "layout.css.stylo-blocklist.enabled" pref to true by default. We can set "layout.css.stylo-blocklist.enabled" pref to true by the styloblocklist system add-on once the "layout.css.stylo-blocklist.blocked_domains" pref is non-empty. MozReview-Commit-ID: 2B5JnGEafLo
modules/libpref/init/all.js
--- a/modules/libpref/init/all.js
+++ b/modules/libpref/init/all.js
@@ -5805,17 +5805,18 @@ pref("dom.webkitBlink.dirPicker.enabled"
 pref("dom.webkitBlink.filesystem.enabled", true);
 #endif
 
 pref("media.block-autoplay-until-in-foreground", true);
 
 // Is Stylo CSS support built and enabled?
 // Only define these prefs if Stylo support is actually built in.
 #ifdef MOZ_STYLO
-pref("layout.css.stylo-blocklist.enabled", true);
+// XXX: We should flip this pref to true once the blocked_domains is non-empty.
+pref("layout.css.stylo-blocklist.enabled", false);
 pref("layout.css.stylo-blocklist.blocked_domains", "");
 #ifdef MOZ_STYLO_ENABLE
 pref("layout.css.servo.enabled", true);
 #else
 pref("layout.css.servo.enabled", false);
 #endif
 #endif