Remove unused ScopedFakeClock from packet_router_unittest This was unused for these tests and passes without it. Bug: webrtc:42223992 Change-Id: Id19b1d7886a0527d49c2f4c30b947b486a6a6964 Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/457581 Reviewed-by: Erik Språng <sprang@webrtc.org> Auto-Submit: Evan Shrubsole <eshr@webrtc.org> Commit-Queue: Erik Språng <sprang@webrtc.org> Cr-Commit-Position: refs/heads/main@{#47205}
diff --git a/modules/pacing/packet_router_unittest.cc b/modules/pacing/packet_router_unittest.cc index ee85329..d0143c2 100644 --- a/modules/pacing/packet_router_unittest.cc +++ b/modules/pacing/packet_router_unittest.cc
@@ -20,7 +20,6 @@ #include "api/rtp_headers.h" #include "api/transport/network_types.h" #include "api/units/data_size.h" -#include "api/units/time_delta.h" #include "modules/rtp_rtcp/include/rtp_header_extension_map.h" #include "modules/rtp_rtcp/include/rtp_rtcp_defines.h" #include "modules/rtp_rtcp/mocks/mock_rtp_rtcp.h" @@ -29,7 +28,6 @@ #include "modules/rtp_rtcp/source/rtp_header_extensions.h" #include "modules/rtp_rtcp/source/rtp_packet_to_send.h" #include "rtc_base/checks.h" -#include "rtc_base/fake_clock.h" #include "test/gmock.h" #include "test/gtest.h" @@ -676,7 +674,6 @@ } TEST(PacketRouterRembTest, ChangeSendRtpModuleChangeRembSender) { - ScopedFakeClock clock; NiceMock<MockRtpRtcpInterface> rtp_send; NiceMock<MockRtpRtcpInterface> rtp_recv; PacketRouter packet_router; @@ -701,7 +698,6 @@ // Only register receiving modules and make sure we fallback to trigger a REMB // packet on this one. TEST(PacketRouterRembTest, NoSendingRtpModule) { - ScopedFakeClock clock; NiceMock<MockRtpRtcpInterface> rtp; PacketRouter packet_router; @@ -722,7 +718,6 @@ } TEST(PacketRouterRembTest, NonCandidateSendRtpModuleNotUsedForRemb) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> module; @@ -740,7 +735,6 @@ } TEST(PacketRouterRembTest, CandidateSendRtpModuleUsedForRemb) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> module; @@ -758,7 +752,6 @@ } TEST(PacketRouterRembTest, NonCandidateReceiveRtpModuleNotUsedForRemb) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> module; @@ -776,7 +769,6 @@ } TEST(PacketRouterRembTest, CandidateReceiveRtpModuleUsedForRemb) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> module; @@ -795,7 +787,6 @@ TEST(PacketRouterRembTest, SendCandidatePreferredOverReceiveCandidate_SendModuleAddedFirst) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> send_module; NiceMock<MockRtpRtcpInterface> receive_module; @@ -822,7 +813,6 @@ TEST(PacketRouterRembTest, SendCandidatePreferredOverReceiveCandidate_ReceiveModuleAddedFirst) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> send_module; NiceMock<MockRtpRtcpInterface> receive_module; @@ -840,7 +830,6 @@ EXPECT_CALL(send_module, SetRemb(bitrate_estimate, ssrcs)); EXPECT_CALL(receive_module, SetRemb(_, _)).Times(0); - clock.AdvanceTime(TimeDelta::Millis(1000)); packet_router.SendRemb(bitrate_estimate, ssrcs); // Test tear-down @@ -849,7 +838,6 @@ } TEST(PacketRouterRembTest, ReceiveModuleTakesOverWhenLastSendModuleRemoved) { - ScopedFakeClock clock; PacketRouter packet_router; NiceMock<MockRtpRtcpInterface> send_module; NiceMock<MockRtpRtcpInterface> receive_module;