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}

modules/video_coding/frame_buffer2_unittest.cc

diff --git a/modules/video_coding/frame_buffer2_unittest.cc b/modules/video_coding/frame_buffer2_unittest.cc
index 2c21a86..13ad66c 100644
--- a/modules/video_coding/frame_buffer2_unittest.cc
+++ b/modules/video_coding/frame_buffer2_unittest.cc
@@ -194,7 +194,7 @@
                          });
     });
     if (max_wait_time == 0) {
-      time_controller_.AdvanceTime(TimeDelta::Millis(0));
+      time_controller_.AdvanceTime(TimeDelta::Zero());
     }
   }
 
@@ -304,7 +304,7 @@
   ExtractFrame(50);
   InsertFrame(pid, 1, ts, true, kFrameSize);
   InsertFrame(pid, 0, ts, false, kFrameSize);
-  time_controller_.AdvanceTime(TimeDelta::Millis(0));
+  time_controller_.AdvanceTime(TimeDelta::Zero());
 
   CheckFrame(0, pid, 0);
   CheckFrame(1, pid, 1);

modules/video_coding/timing/timing_unittest.cc

diff --git a/modules/video_coding/timing/timing_unittest.cc b/modules/video_coding/timing/timing_unittest.cc
index c12de99..8633c0d 100644
--- a/modules/video_coding/timing/timing_unittest.cc
+++ b/modules/video_coding/timing/timing_unittest.cc
@@ -150,10 +150,10 @@
   timing.set_max_playout_delay(TimeDelta::Millis(20));
   EXPECT_FALSE(timing.RenderParameters().use_low_latency_rendering);
   // True if min==0, max > 0.
-  timing.set_min_playout_delay(TimeDelta::Millis(0));
+  timing.set_min_playout_delay(TimeDelta::Zero());
   EXPECT_TRUE(timing.RenderParameters().use_low_latency_rendering);
   // True if min==max==0.
-  timing.set_max_playout_delay(TimeDelta::Millis(0));
+  timing.set_max_playout_delay(TimeDelta::Zero());
   EXPECT_TRUE(timing.RenderParameters().use_low_latency_rendering);
   // True also for max playout delay==500 ms.
   timing.set_max_playout_delay(TimeDelta::Millis(500));