From f783276fb96ee33f23854677970c73d4a593b3ad Mon Sep 17 00:00:00 2001
From: Razvan Maries <rmaries@mozilla.com>
Date: Fri, 31 Jan 2020 17:04:10 +0200
Subject: [PATCH] Backed out changeset fe27999955a4 (bug 1610836) for multiple
 perma failures. CLOSED TREE

--HG--
extra : amend_source : 7f648c73ebfbd227abf477d975f033e507cb151b
---
 netwerk/dns/nsHostResolver.cpp |  5 +----
 netwerk/test/unit/test_trr.js  | 23 -----------------------
 2 files changed, 1 insertion(+), 27 deletions(-)

diff --git a/netwerk/dns/nsHostResolver.cpp b/netwerk/dns/nsHostResolver.cpp
index 42e2ad384778..b3215212764f 100644
--- a/netwerk/dns/nsHostResolver.cpp
+++ b/netwerk/dns/nsHostResolver.cpp
@@ -1453,12 +1453,9 @@ nsresult nsHostResolver::NameLookup(nsHostRecord* rec) {
     rv = TrrLookup(rec);
   }
 
-  bool serviceNotReady =
-      !gTRRService || !gTRRService->Enabled(effectiveRequestMode);
-
   if (effectiveRequestMode == nsIRequest::TRR_DISABLED_MODE ||
       (effectiveRequestMode == nsIRequest::TRR_FIRST_MODE &&
-       (rec->flags & RES_DISABLE_TRR || serviceNotReady) && NS_FAILED(rv))) {
+       (rec->flags & RES_DISABLE_TRR) && NS_FAILED(rv))) {
     if (!rec->IsAddrRecord()) {
       return rv;
     }
diff --git a/netwerk/test/unit/test_trr.js b/netwerk/test/unit/test_trr.js
index af789cf28e52..2f676f034db1 100644
--- a/netwerk/test/unit/test_trr.js
+++ b/netwerk/test/unit/test_trr.js
@@ -1377,26 +1377,3 @@ add_task(async function test_vpnDetection() {
     "changed"
   );
 });
-
-// confirmationNS set without confirmed NS yet
-// checks that we properly fall back to DNS is confirmation is not ready yet
-add_task(async function test_resolve_not_confirmed() {
-  dns.clearCache(true);
-  Services.prefs.setIntPref("network.trr.mode", 2); // TRR-first
-  Services.prefs.clearUserPref("network.trr.useGET");
-  Services.prefs.clearUserPref("network.trr.disable-ECS");
-  Services.prefs.setCharPref(
-    "network.trr.uri",
-    `https://foo.example.com:${h2Port}/doh?responseIP=1::ffff`
-  );
-  Services.prefs.setCharPref(
-    "network.trr.confirmationNS",
-    "confirm.example.com"
-  );
-
-  let [, , inStatus] = await new DNSListener("example.org", undefined, false);
-  Assert.ok(
-    Components.isSuccessCode(inStatus),
-    `${inStatus} should be a success code`
-  );
-});