commit a204ad210df71bef884e13af71dc68553bd39e79
author Philipp Hancke <philipp.hancke@googlemail.com> Fri Jul 08 18:43:25 2022 +0200
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue Aug 02 13:52:36 2022 +0000
tree e70d6220d983a9502a12e78edc3913eb0146ad02
parent 175d77ce77582b34f847535af5435ba0f792c05f

clean up misc TimeDelta use

follow-up from https://webrtc-review.googlesource.com/c/src/+/262810

* replace Time::Millis(0) and TimeDelta::Millis(0) with ::Zero()
* drop unnecessary webrtc namespace from some TimeDeltas
* make TimeDelta do the unit conversion for stats

BUG=webrtc:13756

Change-Id: Ic60625ae0fc7959a47a6be9f5051851feaf76373
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/265875
Reviewed-by: Danil Chapovalov <danilchap@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Commit-Queue: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#37664}

api/rtp_packet_info_unittest.cc

diff --git a/api/rtp_packet_info_unittest.cc b/api/rtp_packet_info_unittest.cc
index 601d34f..0e82aef 100644
--- a/api/rtp_packet_info_unittest.cc
+++ b/api/rtp_packet_info_unittest.cc
@@ -37,7 +37,7 @@
   rhs = RtpPacketInfo();
   EXPECT_NE(rhs.ssrc(), value);
 
-  rhs = RtpPacketInfo(value, {}, {}, {}, {}, Timestamp::Millis(0));
+  rhs = RtpPacketInfo(value, {}, {}, {}, {}, Timestamp::Zero());
   EXPECT_EQ(rhs.ssrc(), value);
 }
 
@@ -64,7 +64,7 @@
   rhs = RtpPacketInfo();
   EXPECT_NE(rhs.csrcs(), value);
 
-  rhs = RtpPacketInfo({}, value, {}, {}, {}, Timestamp::Millis(0));
+  rhs = RtpPacketInfo({}, value, {}, {}, {}, Timestamp::Zero());
   EXPECT_EQ(rhs.csrcs(), value);
 }
 
@@ -91,7 +91,7 @@
   rhs = RtpPacketInfo();
   EXPECT_NE(rhs.rtp_timestamp(), value);
 
-  rhs = RtpPacketInfo({}, {}, value, {}, {}, Timestamp::Millis(0));
+  rhs = RtpPacketInfo({}, {}, value, {}, {}, Timestamp::Zero());
   EXPECT_EQ(rhs.rtp_timestamp(), value);
 }
 
@@ -118,7 +118,7 @@
   rhs = RtpPacketInfo();
   EXPECT_NE(rhs.audio_level(), value);
 
-  rhs = RtpPacketInfo({}, {}, {}, value, {}, Timestamp::Millis(0));
+  rhs = RtpPacketInfo({}, {}, {}, value, {}, Timestamp::Zero());
   EXPECT_EQ(rhs.audio_level(), value);
 }
 
@@ -145,7 +145,7 @@
   rhs = RtpPacketInfo();
   EXPECT_NE(rhs.absolute_capture_time(), value);
 
-  rhs = RtpPacketInfo({}, {}, {}, {}, value, Timestamp::Millis(0));
+  rhs = RtpPacketInfo({}, {}, {}, {}, value, Timestamp::Zero());
   EXPECT_EQ(rhs.absolute_capture_time(), value);
 }
 
@@ -174,7 +174,7 @@
 
   // Default local capture clock offset is null.
   rhs = RtpPacketInfo({}, {}, {}, {}, AbsoluteCaptureTime{12, 34},
-                      Timestamp::Millis(0));
+                      Timestamp::Zero());
   EXPECT_EQ(rhs.local_capture_clock_offset(), absl::nullopt);
 }