Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(620)

Unified Diff: webrtc/base/timeutils_unittest.cc

Issue 2514553003: Change rtc::TimeNanos and rtc::TimeMicros return value from uint64_t to int64_t. (Closed)
Patch Set: Rebased, on top of fixed BoringSSL TimeCallback.' Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « webrtc/base/timeutils.cc ('k') | webrtc/modules/audio_device/audio_device_buffer.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: webrtc/base/timeutils_unittest.cc
diff --git a/webrtc/base/timeutils_unittest.cc b/webrtc/base/timeutils_unittest.cc
index 6707d8c440c40ded212fdb2930ca1219fa125e50..db939cb68bd41a28480af7c560d8f7dceb4bbf7f 100644
--- a/webrtc/base/timeutils_unittest.cc
+++ b/webrtc/base/timeutils_unittest.cc
@@ -303,11 +303,11 @@ TEST(FakeClock, TimeFunctionsUseFakeClock) {
FakeClock clock;
SetClockForTesting(&clock);
- clock.SetTimeNanos(987654321u);
+ clock.SetTimeNanos(987654321);
EXPECT_EQ(987u, Time32());
EXPECT_EQ(987, TimeMillis());
- EXPECT_EQ(987654u, TimeMicros());
- EXPECT_EQ(987654321u, TimeNanos());
+ EXPECT_EQ(987654, TimeMicros());
+ EXPECT_EQ(987654321, TimeNanos());
EXPECT_EQ(1000u, TimeAfter(13));
SetClockForTesting(nullptr);
@@ -317,27 +317,27 @@ TEST(FakeClock, TimeFunctionsUseFakeClock) {
TEST(FakeClock, InitialTime) {
FakeClock clock;
- EXPECT_EQ(0u, clock.TimeNanos());
+ EXPECT_EQ(0, clock.TimeNanos());
}
TEST(FakeClock, SetTimeNanos) {
FakeClock clock;
- clock.SetTimeNanos(123u);
- EXPECT_EQ(123u, clock.TimeNanos());
- clock.SetTimeNanos(456u);
- EXPECT_EQ(456u, clock.TimeNanos());
+ clock.SetTimeNanos(123);
+ EXPECT_EQ(123, clock.TimeNanos());
+ clock.SetTimeNanos(456);
+ EXPECT_EQ(456, clock.TimeNanos());
}
TEST(FakeClock, AdvanceTime) {
FakeClock clock;
clock.AdvanceTime(TimeDelta::FromNanoseconds(1111u));
- EXPECT_EQ(1111u, clock.TimeNanos());
+ EXPECT_EQ(1111, clock.TimeNanos());
clock.AdvanceTime(TimeDelta::FromMicroseconds(2222u));
- EXPECT_EQ(2223111u, clock.TimeNanos());
+ EXPECT_EQ(2223111, clock.TimeNanos());
clock.AdvanceTime(TimeDelta::FromMilliseconds(3333u));
- EXPECT_EQ(3335223111u, clock.TimeNanos());
+ EXPECT_EQ(3335223111, clock.TimeNanos());
clock.AdvanceTime(TimeDelta::FromSeconds(4444u));
- EXPECT_EQ(4447335223111u, clock.TimeNanos());
+ EXPECT_EQ(4447335223111, clock.TimeNanos());
}
// When the clock is advanced, threads that are waiting in a socket select
« no previous file with comments | « webrtc/base/timeutils.cc ('k') | webrtc/modules/audio_device/audio_device_buffer.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698