dcc2321e12
Often wait condition can be met by running pending tasks from other thread and tasks queues without advancing time. while changing TimeController::Wait, relax condition parameter to use view type as function doesn't need to be copied or moved to another context. Bug: None Change-Id: Iaf814dd6e35050becefbbb455a1ef9b46244e89d Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/415500 Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org> Commit-Queue: Danil Chapovalov <danilchap@webrtc.org> Cr-Commit-Position: refs/heads/main@{#45892}
59 lines
1.9 KiB
C++
59 lines
1.9 KiB
C++
/*
|
|
* Copyright (c) 2020 The WebRTC project authors. All Rights Reserved.
|
|
*
|
|
* Use of this source code is governed by a BSD-style license
|
|
* that can be found in the LICENSE file in the root of the source
|
|
* tree. An additional intellectual property rights grant can be found
|
|
* in the file PATENTS. All contributing project authors may
|
|
* be found in the AUTHORS file in the root of the source tree.
|
|
*/
|
|
#include "api/test/time_controller.h"
|
|
|
|
#include <memory>
|
|
|
|
#include "absl/strings/string_view.h"
|
|
#include "api/function_view.h"
|
|
#include "api/task_queue/task_queue_base.h"
|
|
#include "api/task_queue/task_queue_factory.h"
|
|
#include "api/units/time_delta.h"
|
|
|
|
namespace webrtc {
|
|
std::unique_ptr<TaskQueueFactory> TimeController::CreateTaskQueueFactory() {
|
|
class FactoryWrapper final : public TaskQueueFactory {
|
|
public:
|
|
explicit FactoryWrapper(TaskQueueFactory* inner_factory)
|
|
: inner_(inner_factory) {}
|
|
std::unique_ptr<TaskQueueBase, TaskQueueDeleter> CreateTaskQueue(
|
|
absl::string_view name,
|
|
Priority priority) const override {
|
|
return inner_->CreateTaskQueue(name, priority);
|
|
}
|
|
|
|
private:
|
|
TaskQueueFactory* const inner_;
|
|
};
|
|
return std::make_unique<FactoryWrapper>(GetTaskQueueFactory());
|
|
}
|
|
bool TimeController::Wait(FunctionView<bool()> condition,
|
|
TimeDelta max_duration) {
|
|
if (condition()) {
|
|
return true;
|
|
}
|
|
// Run pending tasks first as they might change result of the `condition` and
|
|
// thus avoid unnecessary advancing time.
|
|
AdvanceTime(TimeDelta::Zero());
|
|
|
|
// 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
|
|
// the system.
|
|
const auto kStep = TimeDelta::Millis(5);
|
|
for (auto elapsed = TimeDelta::Zero(); elapsed < max_duration;
|
|
elapsed += kStep) {
|
|
if (condition())
|
|
return true;
|
|
AdvanceTime(kStep);
|
|
}
|
|
return condition();
|
|
}
|
|
} // namespace webrtc
|