bug 1472662 - only send DNS_TRR_NS_VERIFIED telemetry once r=mcmanus

... by making sure we only retry TRR when we go from CP bad=>good.

MozReview-Commit-ID: FcDwzSHm6Ia

--HG--
extra : rebase_source : dce21e18e6a4d854bd2023c61974658b100c1484

netwerk/dns/TRRService.cpp

@@ -303,9 +303,11 @@ TRRService::Observe(nsISupports *aSubject,
       }
     }
 
-    if (mConfirmationState != CONFIRM_OK) {
-      mConfirmationState = CONFIRM_TRYING;
-      MaybeConfirm();
+    if (!mCaptiveIsPassed) {
+      if (mConfirmationState != CONFIRM_OK) {
+        mConfirmationState = CONFIRM_TRYING;
+        MaybeConfirm();
+      }
     } else {
       LOG(("TRRservice CP clear when already up!\n"));
     }