Bug 1558887: Reduce this STUN response delay a little, since on some platforms it seems to be too long. r=mjf a=RyanVM
authorByron Campen [:bwc] <docfaraday@gmail.com>
Wed, 10 Jul 2019 02:07:19 +0000
changeset 544635 016e96e8f65f8f3665ac328169061290ce8521b4
parent 544634 ec6c6890c684785000c5559c805f308ed427c94c
child 544636 6f91ca9e857f2611a9ba411ba3eafb870c28e04b
push id2131
push userffxbld-merge
push dateMon, 26 Aug 2019 18:30:20 +0000
treeherdermozilla-release@b19ffb3ca153 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersmjf, RyanVM
bugs1558887
milestone69.0
Bug 1558887: Reduce this STUN response delay a little, since on some platforms it seems to be too long. r=mjf a=RyanVM Differential Revision: https://phabricator.services.mozilla.com/D37474
media/mtransport/test/ice_unittest.cpp
--- a/media/mtransport/test/ice_unittest.cpp
+++ b/media/mtransport/test/ice_unittest.cpp
@@ -3197,18 +3197,18 @@ TEST_F(WebRtcIceConnectTest, TestConsent
 }
 
 TEST_F(WebRtcIceConnectTest, TestConsentDelayed) {
   AddStream(1);
   SetupAndCheckConsent();
   /* Note: We don't have a list of STUN transaction IDs of the previously timed
            out consent requests. Thus responses after sending the next consent
            request are ignored. */
-  p1_->SetStunResponseDelay(300);
-  p2_->SetStunResponseDelay(300);
+  p1_->SetStunResponseDelay(200);
+  p2_->SetStunResponseDelay(200);
   PR_Sleep(1000);
   AssertConsentRefresh();
   SendReceive();
 }
 
 TEST_F(WebRtcIceConnectTest, TestNetworkForcedOfflineAndRecovery) {
   AddStream(1);
   SetupAndCheckConsent();