Bug 1944631 - Remove the HTTP_RESPONSE_STATUS_CODE histogram that already exists in Glean, r=acreskey,necko-reviewers,valentin.

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

Author: fqueze

netwerk/protocol/http/nsHttpChannel.cpp

@@ -2564,90 +2564,75 @@ nsresult nsHttpChannel::ProcessResponse() {
     // of auth failures due to Bug 1896350
     switch (httpStatus) {
       case 200:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 0);
         mozilla::glean::networking::http_response_status_code.Get("200_ok"_ns)
             .Add(1);
         break;
       case 301:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 1);
         mozilla::glean::networking::http_response_status_code
             .Get("301_moved_permanently"_ns)
             .Add(1);
         break;
       case 302:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 2);
         mozilla::glean::networking::http_response_status_code
             .Get("302_found"_ns)
             .Add(1);
         break;
       case 304:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 3);
         mozilla::glean::networking::http_response_status_code
             .Get("304_not_modified"_ns)
             .Add(1);
         break;
       case 307:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 4);
         mozilla::glean::networking::http_response_status_code
             .Get("307_temporary_redirect"_ns)
             .Add(1);
         break;
       case 308:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 5);
         mozilla::glean::networking::http_response_status_code
             .Get("308_permanent_redirect"_ns)
             .Add(1);
         break;
       case 400:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 6);
         mozilla::glean::networking::http_response_status_code
             .Get("400_bad_request"_ns)
             .Add(1);
         break;
       case 401:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 7);
         mozilla::glean::networking::http_response_status_code
             .Get("401_unauthorized"_ns)
             .Add(1);
         break;
       case 403:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 8);
         mozilla::glean::networking::http_response_status_code
             .Get("403_forbidden"_ns)
             .Add(1);
         break;
       case 404:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 9);
         mozilla::glean::networking::http_response_status_code
             .Get("404_not_found"_ns)
             .Add(1);
         break;
       case 421:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 11);
         mozilla::glean::networking::http_response_status_code
             .Get("421_misdirected_request"_ns)
             .Add(1);
         break;
       case 425:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 11);
         mozilla::glean::networking::http_response_status_code
             .Get("425_too_early"_ns)
             .Add(1);
         break;
       case 429:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 11);
         mozilla::glean::networking::http_response_status_code
             .Get("429_too_many_requests"_ns)
             .Add(1);
         break;
       case 500:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 10);
         mozilla::glean::networking::http_response_status_code
             .Get("other_5xx"_ns)
             .Add(1);
         break;
       default:
-        Telemetry::Accumulate(Telemetry::HTTP_RESPONSE_STATUS_CODE, 11);
         if (httpStatus >= 400 && httpStatus < 500) {
           mozilla::glean::networking::http_response_status_code
               .Get("other_4xx"_ns)

toolkit/components/telemetry/Histograms.json

@@ -2863,16 +2863,6 @@
     "kind": "boolean",
     "description": "Whether or not a new alt-svc mapping would change the target hostname of the existing mapping"
   },
-  "HTTP_RESPONSE_STATUS_CODE": {
-    "record_in_processes": ["main", "content"],
-    "products": ["firefox", "fennec"],
-    "alert_emails": ["smayya@mozilla.com"],
-    "bug_numbers": [1896350],
-    "expires_in_version": "never",
-    "kind": "enumerated",
-    "n_values": 12,
-    "description": "Http response status (0=200, 1=301, 2=302, 3=304, 4=307, 5=308, 6=400, 7=401, 8=403, 9=404, 10=500, 11=other)"
-  },
   "HTTP_ONSTART_SUSPEND_TOTAL_TIME": {
     "record_in_processes": ["main", "content"],
     "products": ["firefox", "fennec"],