Bug 1198345 - Split moar Hello Telemetry values from general WebRTC. r=jesup

This commit is contained in:
Jan-Ivar Bruaroey 2015-12-10 23:38:02 -05:00
parent 96471b7cc5
commit 4ef0163968
2 changed files with 20 additions and 2 deletions

View File

@ -1961,10 +1961,14 @@ PeerConnectionImpl::AddIceCandidate(const char* aCandidate, const char* aMid, un
if(!mIceStartTime.IsNull()) {
TimeDuration timeDelta = TimeStamp::Now() - mIceStartTime;
if (mIceConnectionState == PCImplIceConnectionState::Failed) {
Telemetry::Accumulate(Telemetry::WEBRTC_ICE_LATE_TRICKLE_ARRIVAL_TIME,
Telemetry::Accumulate((mIsLoop ?
Telemetry::LOOP_ICE_LATE_TRICKLE_ARRIVAL_TIME :
Telemetry::WEBRTC_ICE_LATE_TRICKLE_ARRIVAL_TIME),
timeDelta.ToMilliseconds());
} else {
Telemetry::Accumulate(Telemetry::WEBRTC_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME,
Telemetry::Accumulate((mIsLoop ?
Telemetry::LOOP_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME :
Telemetry::WEBRTC_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME),
timeDelta.ToMilliseconds());
}
}

View File

@ -9531,6 +9531,20 @@
"n_values": 7,
"description": "The ICE connection state when the PC was closed"
},
"LOOP_ICE_ON_TIME_TRICKLE_ARRIVAL_TIME": {
"expires_in_version": "never",
"kind": "exponential",
"high": "10000",
"n_buckets": "20",
"description": "The length of time (in milliseconds) that a trickle candidate took to arrive after the start of ICE, given that it arrived when ICE was not in a failure state (ie; a candidate that we could do something with, hence 'on time')"
},
"LOOP_ICE_LATE_TRICKLE_ARRIVAL_TIME": {
"expires_in_version": "never",
"kind": "exponential",
"high": "10000",
"n_buckets": "20",
"description": "The length of time (in milliseconds) that a trickle candidate took to arrive after the start of ICE, given that it arrived after ICE failed."
},
"LOOP_ICE_SUCCESS_TIME": {
"expires_in_version": "never",
"kind": "exponential",