Rename done() into condition(), because it is actually condition in TimeController API
Bug: None
Change-Id: Ia3a742d1d2ad1238223f4da7ae843a8d22108ec5
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/174060
Reviewed-by: Sebastian Jansson <srte@webrtc.org>
Commit-Queue: Artem Titov <titovartem@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31144}
diff --git a/api/test/time_controller.cc b/api/test/time_controller.cc
index f2b1a05..364dbc2 100644
--- a/api/test/time_controller.cc
+++ b/api/test/time_controller.cc
@@ -26,7 +26,7 @@
};
return std::make_unique<FactoryWrapper>(GetTaskQueueFactory());
}
-bool TimeController::Wait(const std::function<bool()>& done,
+bool TimeController::Wait(const std::function<bool()>& condition,
TimeDelta max_duration) {
// Step size is chosen to be short enough to not significantly affect latency
// in real time tests while being long enough to avoid adding too much load to
@@ -34,10 +34,10 @@
const auto kStep = TimeDelta::Millis(5);
for (auto elapsed = TimeDelta::Zero(); elapsed < max_duration;
elapsed += kStep) {
- if (done())
+ if (condition())
return true;
AdvanceTime(kStep);
}
- return done();
+ return condition();
}
} // namespace webrtc