Bug 1857926: re-add OnStart/StopRequest Telemetry to monitor delay r=…

…acreskey,necko-reviewers,valentin

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

UltraBlame original commit: 1645fbba770b4bcf1ef910d4fb0d18ce2e58cf57

netwerk/metrics.yaml

@@ -911,6 +911,192 @@ never
 telemetry_mirror
 :
 DNS_FAILED_LOOKUP_TIME
+http_content_onstart_delay
+:
+type
+:
+timing_distribution
+time_unit
+:
+millisecond
+description
+:
+>
+The
+time
+between
+dispatching
+OnStartRequest
+from
+the
+socket
+thread
+and
+processing
+it
+on
+the
+main
+thread
+(
+content
+process
+)
+.
+bugs
+:
+-
+https
+:
+/
+/
+bugzilla
+.
+mozilla
+.
+org
+/
+show_bug
+.
+cgi
+?
+id
+=
+1857926
+data_reviews
+:
+-
+https
+:
+/
+/
+bugzilla
+.
+mozilla
+.
+org
+/
+show_bug
+.
+cgi
+?
+id
+=
+1857926
+#
+c
+data_sensitivity
+:
+-
+technical
+notification_emails
+:
+-
+necko
+mozilla
+.
+com
+-
+rjesup
+mozilla
+.
+com
+expires
+:
+130
+http_content_onstop_delay
+:
+type
+:
+timing_distribution
+time_unit
+:
+millisecond
+description
+:
+>
+The
+time
+between
+dispatching
+OnStopRequest
+from
+the
+socket
+thread
+and
+processing
+it
+on
+the
+main
+thread
+(
+content
+process
+)
+.
+bugs
+:
+-
+https
+:
+/
+/
+bugzilla
+.
+mozilla
+.
+org
+/
+show_bug
+.
+cgi
+?
+id
+=
+1857926
+data_reviews
+:
+-
+https
+:
+/
+/
+bugzilla
+.
+mozilla
+.
+org
+/
+show_bug
+.
+cgi
+?
+id
+=
+1857926
+#
+c
+data_sensitivity
+:
+-
+technical
+notification_emails
+:
+-
+necko
+mozilla
+.
+com
+-
+rjesup
+mozilla
+.
+com
+expires
+:
+130
 http_1_download_throughput
 :
 type

netwerk/protocol/http/HttpChannelChild.cpp

@@ -1702,6 +1702,16 @@ OnSocketThread
 )
 )
 ;
+TimeStamp
+start
+=
+TimeStamp
+:
+:
+Now
+(
+)
+;
 mAltDataInputStream
 =
 DeserializeIPCStream
@@ -1736,17 +1746,35 @@ aResponseHead
 aUseResponseHead
 aRequestHeaders
 aArgs
-aOnStartRequestStartTime
+start
 ]
 (
 )
 {
-self
--
->
-mOnStartRequestStartTime
+TimeDuration
+delay
 =
-aOnStartRequestStartTime
+TimeStamp
+:
+:
+Now
+(
+)
+-
+start
+;
+glean
+:
+:
+networking
+:
+:
+http_content_onstart_delay
+.
+AccumulateRawDuration
+(
+delay
+)
 ;
 self
 -
@@ -4218,6 +4246,16 @@ encodedBodySize
 )
 ;
 }
+TimeStamp
+start
+=
+TimeStamp
+:
+:
+Now
+(
+)
+;
 if
 (
 StaticPrefs
@@ -4322,18 +4360,36 @@ Clone
 )
 }
 aFromSocketProcess
-aOnStopRequestStartTime
+start
 ]
 (
 )
 mutable
 {
-self
--
->
-mOnStopRequestStartTime
+TimeDuration
+delay
 =
-aOnStopRequestStartTime
+TimeStamp
+:
+:
+Now
+(
+)
+-
+start
+;
+glean
+:
+:
+networking
+:
+:
+http_content_onstop_delay
+.
+AccumulateRawDuration
+(
+delay
+)
 ;
 self
 -