Bug 1637648 - P5: Create mConfirmer when network.trr.confirmationNS is changed r=valentin
authorKershaw Chang <kershaw@mozilla.com>
Tue, 23 Jun 2020 11:11:49 +0000
changeset 536869 34fb904b1175fc534e40ec02e64824acb056d55f
parent 536868 585801742fa2530c9b27e58bf4cd96d8dff9957a
child 536870 4a59ad84db1b0f7129b2e28c239b1035f277f3af
push id37533
push userdluca@mozilla.com
push dateTue, 23 Jun 2020 21:38:40 +0000
treeherdermozilla-central@d48aa0f0aa0b [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersvalentin
bugs1637648
milestone79.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1637648 - P5: Create mConfirmer when network.trr.confirmationNS is changed r=valentin Differential Revision: https://phabricator.services.mozilla.com/D78887
netwerk/dns/TRRService.cpp
--- a/netwerk/dns/TRRService.cpp
+++ b/netwerk/dns/TRRService.cpp
@@ -275,16 +275,17 @@ nsresult TRRService::ReadPrefs(const cha
   if (!name || !strcmp(name, TRR_PREF("confirmationNS"))) {
     MutexAutoLock lock(mLock);
     nsAutoCString old(mConfirmationNS);
     Preferences::GetCString(TRR_PREF("confirmationNS"), mConfirmationNS);
     if (name && !old.IsEmpty() && !mConfirmationNS.Equals(old) &&
         (mConfirmationState > CONFIRM_TRYING)) {
       LOG(("TRR::ReadPrefs: restart confirmationNS state\n"));
       mConfirmationState = CONFIRM_TRYING;
+      MaybeConfirm_locked();
     }
   }
   if (!name || !strcmp(name, TRR_PREF("bootstrapAddress"))) {
     MutexAutoLock lock(mLock);
     Preferences::GetCString(TRR_PREF("bootstrapAddress"), mBootstrapAddr);
     clearEntireCache = true;
   }
   if (!name || !strcmp(name, TRR_PREF("wait-for-portal"))) {