Bug 1471217 - ignore the network.dns.get-ttl pref on non-supported systems r=mcmanus a=lizzard DEVEDITION_62_0b4_BUILD1 DEVEDITION_62_0b4_RELEASE FIREFOX_62_0b4_BUILD1 FIREFOX_62_0b4_RELEASE
authorDaniel Stenberg <daniel@haxx.se>
Tue, 26 Jun 2018 15:36:26 +0200
changeset 480248 e09909cb2297c14e75e9a94425af8e5b8e605094
parent 480247 7ccdd10cf1b0baeeb77b18894641f140403ccb10
child 480249 e5896ff76fe48c1cfd1960d53c067660f1a8a620
push id1757
push userffxbld-merge
push dateFri, 24 Aug 2018 17:02:43 +0000
treeherdermozilla-release@736023aebdb1 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmcmanus, lizzard
bugs1471217
milestone62.0
Bug 1471217 - ignore the network.dns.get-ttl pref on non-supported systems r=mcmanus a=lizzard Which is how the pref is documented in all.js MozReview-Commit-ID: 5ONQgZ1wO5G
netwerk/dns/nsHostResolver.cpp
--- a/netwerk/dns/nsHostResolver.cpp
+++ b/netwerk/dns/nsHostResolver.cpp
@@ -503,17 +503,19 @@ static void DnsPrefChanged(const char* a
 {
     MOZ_ASSERT(NS_IsMainThread(),
                "Should be getting pref changed notification on main thread!");
 
     DebugOnly<nsHostResolver*> self = static_cast<nsHostResolver*>(aClosure);
     MOZ_ASSERT(self);
 
     if (!strcmp(aPref, kPrefGetTtl)) {
+#ifdef DNSQUERY_AVAILABLE
         sGetTtlEnabled = Preferences::GetBool(kPrefGetTtl);
+#endif
     } else if (!strcmp(aPref, kPrefNativeIsLocalhost)) {
         gNativeIsLocalhost = Preferences::GetBool(kPrefNativeIsLocalhost);
     }
 }
 
 NS_IMPL_ISUPPORTS0(nsHostResolver)
 
 nsHostResolver::nsHostResolver(uint32_t maxCacheEntries,