Bug 1720614 - Remove HTTPS_RR_WAITING_TIME, r=necko-reviewers,valentin

Differential Revision: https://phabricator.services.mozilla.com/D122123

netwerk/protocol/http/nsHttpTransaction.cpp

@@ -373,7 +373,6 @@ nsresult nsHttpTransaction::Init(
     if (target) {
       if (StaticPrefs::network_dns_force_waiting_https_rr()) {
         mCaps |= NS_HTTP_FORCE_WAIT_HTTP_RR;
-        mHTTPSRRQueryStart = TimeStamp::Now();
       }
 
       mResolver = new HTTPSRecordResolver(this);
@@ -3034,14 +3033,6 @@ nsresult nsHttpTransaction::OnHTTPSRRAvailable(
   auto updateHTTPSSVCReceivedStage = MakeScopeExit([&] {
     mHTTPSSVCReceivedStage = receivedStage;
 
-    if (!mHTTPSRRQueryStart.IsNull()) {
-      AccumulateTimeDelta(Telemetry::HTTPS_RR_WAITING_TIME,
-                          HTTPS_RR_IS_USED(mHTTPSSVCReceivedStage)
-                              ? "with_https_rr"_ns
-                              : "no_https_rr"_ns,
-                          mHTTPSRRQueryStart, TimeStamp::Now());
-    }
-
     // In the case that an HTTPS RR is unavailable, we should call
     // ProcessPendingQ to make sure this transition to be processed soon.
     if (!mHTTPSSVCRecord) {

netwerk/protocol/http/nsHttpTransaction.h

@@ -440,7 +440,6 @@ class nsHttpTransaction final : public nsAHttpTransaction,
 
   // The time when the transaction was submitted to the Connection Manager
   TimeStamp mPendingTime;
-  TimeStamp mHTTPSRRQueryStart;
   TimeDuration mPendingDurationTime;
 
   uint64_t mTopBrowsingContextId{0};

toolkit/components/telemetry/Histograms.json

@@ -16645,17 +16645,5 @@
       "Fetch",
       "Unexpected"
     ]
-  },
-  "HTTPS_RR_WAITING_TIME": {
-    "record_in_processes": ["main", "socket"],
-    "products": ["firefox"],
-    "alert_emails": ["[email protected]", "[email protected]"],
-    "bug_numbers": [1714506],
-    "expires_in_version": "93",
-    "kind": "exponential",
-    "high": 5000,
-    "n_buckets": 100,
-    "keyed": true,
-    "description": "The time that a transaction is waiting for HTTPS RR"
   }
 }