tor-browser

commit 52296e3525e6eb23533e60cd69df3a0542c51bdf
parent 8976f01f872306fbeeca2cf949fa242711120b85
Author: smayya <smayya@mozilla.com>
Date:   Wed,  1 Oct 2025 07:57:08 +0000

Bug 1989184 -  remove metrics  networking.http_content_cssloader_ondatafinished_to_onstop_delay. r=necko-reviewers,valentin

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

Diffstat:
M
layout/style/StreamLoader.cpp
 | 
11
-----------
M
layout/style/StreamLoader.h
 | 
2
--
M
netwerk/metrics.yaml
 | 
18
------------------

3 files changed, 0 insertions(+), 31 deletions(-)
diff --git a/layout/style/StreamLoader.cpp b/layout/style/StreamLoader.cpp
@@ -10,7 +10,6 @@
 #include "mozilla/StaticPrefs_network.h"
 #include "mozilla/TaskQueue.h"
 #include "mozilla/dom/CacheExpirationTime.h"
-#include "mozilla/glean/NetwerkMetrics.h"
 #include "mozilla/net/UrlClassifierFeatureFactory.h"
 #include "nsContentUtils.h"
 #include "nsIAsyncVerifyRedirectCallback.h"
@@ -106,13 +105,6 @@ StreamLoader::OnStopRequest(nsIRequest* aRequest, nsresult aStatus) {
     mSheetLoadData->mNetworkMetadata =
         new SubResourceNetworkMetadataHolder(aRequest);
 
-    if (mOnDataFinishedTime) {
-      // collect telemetry for the delta between OnDataFinished and
-      // OnStopRequest
-      TimeDuration delta = (TimeStamp::Now() - mOnDataFinishedTime);
-      glean::networking::http_content_cssloader_ondatafinished_to_onstop_delay
-          .AccumulateRawDuration(delta);
-    }
     mSheetLoadData->mSheet->UnblockParsePromise();
   } else {
     if (mSheetLoadData->mRecordErrors) {
@@ -243,9 +235,6 @@ NS_IMETHODIMP
 StreamLoader::OnDataFinished(nsresult aResult) {
   nsCOMPtr<nsIRequest> request = mRequest.forget();
   if (StaticPrefs::network_send_OnDataFinished_cssLoader()) {
-    MOZ_ASSERT(mOnDataFinishedTime.IsNull(),
-               "OnDataFinished should only be called once");
-    mOnDataFinishedTime = TimeStamp::Now();
     return OnStopRequest(request, aResult);
   }
 
diff --git a/layout/style/StreamLoader.h b/layout/style/StreamLoader.h
@@ -64,8 +64,6 @@ class StreamLoader : public nsIThreadRetargetableStreamListener,
   bool mOnStopProcessingDone{false};
   RefPtr<SheetLoadDataHolder> mMainThreadSheetLoadData;
 
-  mozilla::TimeStamp mOnDataFinishedTime;
-
 #ifdef NIGHTLY_BUILD
   bool mChannelOpenFailed = false;
 #endif
diff --git a/netwerk/metrics.yaml b/netwerk/metrics.yaml
@@ -824,24 +824,6 @@ networking:
       - rjesup@mozilla.com
     expires: 146
 
-  http_content_cssloader_ondatafinished_to_onstop_delay:
-    type: timing_distribution
-    time_unit: millisecond
-    description: >
-      The time between processing OnDataFinished and processing OnStopRequest for CSSLoader in the content process
-    bugs:
-      - https://bugzilla.mozilla.org/show_bug.cgi?id=1883391
-      - https://bugzilla.mozilla.org/show_bug.cgi?id=1902186
-    data_reviews:
-      - https://bugzilla.mozilla.org/show_bug.cgi?id=1883391
-    data_sensitivity:
-      - technical
-    notification_emails:
-      - necko@mozilla.com
-    expires: 146
-
-
-
   http_onstart_suspend_total_time:
     type: timing_distribution
     time_unit: millisecond