1572505 - Convert network.dns.disablePrefetchFromHTTPS to use a Static Pref

Status: RESOLVED FIXED

(Core :: Preferences: Backend, task)

Description

[Kris Wright :KrisWright]

network.dns.disablePrefetchFromHTTPS needs to be converted from a VarCache pref to a static pref.

For this particular pref, AddBoolVarCache is called and defaulted to false, but the value then immediately uses true as a fallback, and a quick check tells me that sDisablePrefetchHTTPSPref is set to true: https://searchfox.org/mozilla-central/rev/9ae20497229225cb3fa729a787791f424ff8087b/dom/html/nsHTMLDNSPrefetch.cpp#68

This is not the best practice since code consulting the VarCache will think the feature is disabled, while code that may consult the prefs table will see that the pref doesn't exist, presumably concluding that the feature is not disabled. It appears the pref should actually be set to true as a default when adding it to StaticPrefs.

Comment 1

[Kris Wright :KrisWright]

Attachment: Bug 1572505 - Convert network.dns.disablePrefetchFromHTTPS to static pref.

Converts network.dns.disablePrefetchFromHTTPS to a static pref. Though the original VarCache pref default to false, it looks like the intended value is true. (see comment #0 in bug 1572505)

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/2cdac58bb3a1
Convert network.dns.disablePrefetchFromHTTPS to static pref. r=smaug,njn

Comment 3

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/2cdac58bb3a1