Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2019 The WebRTC Project Authors. All rights reserved. |
| 3 | * |
| 4 | * Use of this source code is governed by a BSD-style license |
| 5 | * that can be found in the LICENSE file in the root of the source |
| 6 | * tree. An additional intellectual property rights grant can be found |
| 7 | * in the file PATENTS. All contributing project authors may |
| 8 | * be found in the AUTHORS file in the root of the source tree. |
| 9 | */ |
| 10 | #include "api/task_queue/task_queue_test.h" |
| 11 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 12 | #include <memory> |
| 13 | |
| 14 | #include "absl/cleanup/cleanup.h" |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 15 | #include "absl/strings/string_view.h" |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 16 | #include "api/task_queue/task_queue_base.h" |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 17 | #include "api/units/time_delta.h" |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 18 | #include "rtc_base/event.h" |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 19 | #include "rtc_base/ref_counter.h" |
Steve Anton | f380284 | 2019-01-25 03:07:40 | [diff] [blame] | 20 | #include "rtc_base/time_utils.h" |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 21 | |
| 22 | namespace webrtc { |
| 23 | namespace { |
| 24 | |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 25 | // Avoids a dependency to system_wrappers. |
| 26 | void SleepFor(TimeDelta duration) { |
| 27 | rtc::ScopedAllowBaseSyncPrimitivesForTesting allow; |
| 28 | rtc::Event event; |
| 29 | event.Wait(duration); |
| 30 | } |
| 31 | |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 32 | std::unique_ptr<TaskQueueBase, TaskQueueDeleter> CreateTaskQueue( |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 33 | const std::unique_ptr<webrtc::TaskQueueFactory>& factory, |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 34 | absl::string_view task_queue_name, |
| 35 | TaskQueueFactory::Priority priority = TaskQueueFactory::Priority::NORMAL) { |
| 36 | return factory->CreateTaskQueue(task_queue_name, priority); |
| 37 | } |
| 38 | |
| 39 | TEST_P(TaskQueueTest, Construct) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 40 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 41 | auto queue = CreateTaskQueue(factory, "Construct"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 42 | EXPECT_FALSE(queue->IsCurrent()); |
| 43 | } |
| 44 | |
| 45 | TEST_P(TaskQueueTest, PostAndCheckCurrent) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 46 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 47 | rtc::Event event; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 48 | auto queue = CreateTaskQueue(factory, "PostAndCheckCurrent"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 49 | |
Artem Titov | 0e61fdd | 2021-07-25 19:50:14 | [diff] [blame] | 50 | // We're not running a task, so `queue` shouldn't be current. |
Tommi | 6866dc7 | 2020-05-15 08:11:56 | [diff] [blame] | 51 | // Note that because rtc::Thread also supports the TQ interface and |
| 52 | // TestMainImpl::Init wraps the main test thread (bugs.webrtc.org/9714), that |
| 53 | // means that TaskQueueBase::Current() will still return a valid value. |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 54 | EXPECT_FALSE(queue->IsCurrent()); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 55 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 56 | queue->PostTask([&event, &queue] { |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 57 | EXPECT_TRUE(queue->IsCurrent()); |
| 58 | event.Set(); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 59 | }); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 60 | EXPECT_TRUE(event.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 61 | } |
| 62 | |
| 63 | TEST_P(TaskQueueTest, PostCustomTask) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 64 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 65 | rtc::Event ran; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 66 | auto queue = CreateTaskQueue(factory, "PostCustomImplementation"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 67 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 68 | class CustomTask { |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 69 | public: |
| 70 | explicit CustomTask(rtc::Event* ran) : ran_(ran) {} |
| 71 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 72 | void operator()() { ran_->Set(); } |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 73 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 74 | private: |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 75 | rtc::Event* const ran_; |
| 76 | } my_task(&ran); |
| 77 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 78 | queue->PostTask(my_task); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 79 | EXPECT_TRUE(ran.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 80 | } |
| 81 | |
| 82 | TEST_P(TaskQueueTest, PostDelayedZero) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 83 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 84 | rtc::Event event; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 85 | auto queue = CreateTaskQueue(factory, "PostDelayedZero"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 86 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 87 | queue->PostDelayedTask([&event] { event.Set(); }, TimeDelta::Zero()); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 88 | EXPECT_TRUE(event.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 89 | } |
| 90 | |
| 91 | TEST_P(TaskQueueTest, PostFromQueue) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 92 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 93 | rtc::Event event; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 94 | auto queue = CreateTaskQueue(factory, "PostFromQueue"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 95 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 96 | queue->PostTask( |
| 97 | [&event, &queue] { queue->PostTask([&event] { event.Set(); }); }); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 98 | EXPECT_TRUE(event.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 99 | } |
| 100 | |
| 101 | TEST_P(TaskQueueTest, PostDelayed) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 102 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 103 | rtc::Event event; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 104 | auto queue = |
| 105 | CreateTaskQueue(factory, "PostDelayed", TaskQueueFactory::Priority::HIGH); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 106 | |
| 107 | int64_t start = rtc::TimeMillis(); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 108 | queue->PostDelayedTask( |
| 109 | [&event, &queue] { |
| 110 | EXPECT_TRUE(queue->IsCurrent()); |
| 111 | event.Set(); |
| 112 | }, |
| 113 | TimeDelta::Millis(100)); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 114 | EXPECT_TRUE(event.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 115 | int64_t end = rtc::TimeMillis(); |
| 116 | // These tests are a little relaxed due to how "powerful" our test bots can |
| 117 | // be. Most recently we've seen windows bots fire the callback after 94-99ms, |
| 118 | // which is why we have a little bit of leeway backwards as well. |
| 119 | EXPECT_GE(end - start, 90u); |
| 120 | EXPECT_NEAR(end - start, 190u, 100u); // Accept 90-290. |
| 121 | } |
| 122 | |
| 123 | TEST_P(TaskQueueTest, PostMultipleDelayed) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 124 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 125 | auto queue = CreateTaskQueue(factory, "PostMultipleDelayed"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 126 | |
| 127 | std::vector<rtc::Event> events(100); |
| 128 | for (int i = 0; i < 100; ++i) { |
| 129 | rtc::Event* event = &events[i]; |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 130 | queue->PostDelayedTask( |
| 131 | [event, &queue] { |
| 132 | EXPECT_TRUE(queue->IsCurrent()); |
| 133 | event->Set(); |
| 134 | }, |
| 135 | TimeDelta::Millis(i)); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 136 | } |
| 137 | |
| 138 | for (rtc::Event& e : events) |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 139 | EXPECT_TRUE(e.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 140 | } |
| 141 | |
| 142 | TEST_P(TaskQueueTest, PostDelayedAfterDestruct) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 143 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 144 | rtc::Event run; |
| 145 | rtc::Event deleted; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 146 | auto queue = CreateTaskQueue(factory, "PostDelayedAfterDestruct"); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 147 | absl::Cleanup cleanup = [&deleted] { deleted.Set(); }; |
| 148 | queue->PostDelayedTask([&run, cleanup = std::move(cleanup)] { run.Set(); }, |
| 149 | TimeDelta::Millis(100)); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 150 | // Destroy the queue. |
| 151 | queue = nullptr; |
| 152 | // Task might outlive the TaskQueue, but still should be deleted. |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 153 | EXPECT_TRUE(deleted.Wait(TimeDelta::Seconds(1))); |
| 154 | EXPECT_FALSE(run.Wait(TimeDelta::Zero())); // and should not run. |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 155 | } |
| 156 | |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 157 | TEST_P(TaskQueueTest, PostDelayedHighPrecisionAfterDestruct) { |
| 158 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
| 159 | rtc::Event run; |
| 160 | rtc::Event deleted; |
| 161 | auto queue = |
| 162 | CreateTaskQueue(factory, "PostDelayedHighPrecisionAfterDestruct"); |
| 163 | absl::Cleanup cleanup = [&deleted] { deleted.Set(); }; |
| 164 | queue->PostDelayedHighPrecisionTask( |
| 165 | [&run, cleanup = std::move(cleanup)] { run.Set(); }, |
| 166 | TimeDelta::Millis(100)); |
| 167 | // Destroy the queue. |
| 168 | queue = nullptr; |
| 169 | // Task might outlive the TaskQueue, but still should be deleted. |
| 170 | EXPECT_TRUE(deleted.Wait(TimeDelta::Seconds(1))); |
| 171 | EXPECT_FALSE(run.Wait(TimeDelta::Zero())); // and should not run. |
| 172 | } |
| 173 | |
| 174 | TEST_P(TaskQueueTest, PostedUnexecutedClosureDestroyedOnTaskQueue) { |
| 175 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
| 176 | auto queue = |
| 177 | CreateTaskQueue(factory, "PostedUnexecutedClosureDestroyedOnTaskQueue"); |
| 178 | TaskQueueBase* queue_ptr = queue.get(); |
| 179 | queue->PostTask([] { SleepFor(TimeDelta::Millis(100)); }); |
Per K | 18aba66 | 2023-06-16 13:42:00 | [diff] [blame] | 180 | // Give the task queue a chance to start executing the first lambda. |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 181 | SleepFor(TimeDelta::Millis(10)); |
Per K | 18aba66 | 2023-06-16 13:42:00 | [diff] [blame] | 182 | rtc::Event finished; |
| 183 | // Then ensure the next lambda (which is likely not executing yet) is |
| 184 | // destroyed in the task queue context when the queue is deleted. |
| 185 | auto cleanup = absl::Cleanup([queue_ptr, &finished] { |
| 186 | EXPECT_EQ(queue_ptr, TaskQueueBase::Current()); |
| 187 | finished.Set(); |
| 188 | }); |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 189 | queue->PostTask([cleanup = std::move(cleanup)] {}); |
| 190 | queue = nullptr; |
Per K | 18aba66 | 2023-06-16 13:42:00 | [diff] [blame] | 191 | finished.Wait(TimeDelta::Seconds(1)); |
| 192 | } |
| 193 | |
| 194 | TEST_P(TaskQueueTest, PostedClosureDestroyedOnTaskQueue) { |
| 195 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
| 196 | auto queue = CreateTaskQueue(factory, "PostedClosureDestroyedOnTaskQueue"); |
| 197 | TaskQueueBase* queue_ptr = queue.get(); |
| 198 | rtc::Event finished; |
| 199 | auto cleanup = absl::Cleanup([queue_ptr, &finished] { |
| 200 | EXPECT_EQ(queue_ptr, TaskQueueBase::Current()); |
| 201 | finished.Set(); |
| 202 | }); |
| 203 | // The cleanup task may or may not have had time to execute when the task |
| 204 | // queue is destroyed. Regardless, the task should be destroyed on the |
| 205 | // queue. |
| 206 | queue->PostTask([cleanup = std::move(cleanup)] {}); |
| 207 | queue = nullptr; |
| 208 | finished.Wait(TimeDelta::Seconds(1)); |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 209 | } |
| 210 | |
| 211 | TEST_P(TaskQueueTest, PostedExecutedClosureDestroyedOnTaskQueue) { |
| 212 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
| 213 | auto queue = |
| 214 | CreateTaskQueue(factory, "PostedExecutedClosureDestroyedOnTaskQueue"); |
| 215 | TaskQueueBase* queue_ptr = queue.get(); |
| 216 | // Ensure an executed lambda is destroyed on the task queue. |
| 217 | rtc::Event finished; |
| 218 | queue->PostTask([cleanup = absl::Cleanup([queue_ptr, &finished] { |
| 219 | EXPECT_EQ(queue_ptr, TaskQueueBase::Current()); |
| 220 | finished.Set(); |
| 221 | })] {}); |
Per K | 18aba66 | 2023-06-16 13:42:00 | [diff] [blame] | 222 | finished.Wait(TimeDelta::Seconds(1)); |
Markus Handell | 82da932 | 2022-12-16 14:50:24 | [diff] [blame] | 223 | } |
| 224 | |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 225 | TEST_P(TaskQueueTest, PostAndReuse) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 226 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 227 | rtc::Event event; |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 228 | auto post_queue = CreateTaskQueue(factory, "PostQueue"); |
| 229 | auto reply_queue = CreateTaskQueue(factory, "ReplyQueue"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 230 | |
| 231 | int call_count = 0; |
| 232 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 233 | class ReusedTask { |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 234 | public: |
| 235 | ReusedTask(int* counter, TaskQueueBase* reply_queue, rtc::Event* event) |
| 236 | : counter_(*counter), reply_queue_(reply_queue), event_(*event) { |
| 237 | EXPECT_EQ(counter_, 0); |
| 238 | } |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 239 | ReusedTask(ReusedTask&&) = default; |
| 240 | ReusedTask& operator=(ReusedTask&&) = delete; |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 241 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 242 | void operator()() && { |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 243 | if (++counter_ == 1) { |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 244 | reply_queue_->PostTask(std::move(*this)); |
| 245 | // At this point, the object is in the moved-from state. |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 246 | } else { |
| 247 | EXPECT_EQ(counter_, 2); |
| 248 | EXPECT_TRUE(reply_queue_->IsCurrent()); |
| 249 | event_.Set(); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 250 | } |
| 251 | } |
| 252 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 253 | private: |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 254 | int& counter_; |
| 255 | TaskQueueBase* const reply_queue_; |
| 256 | rtc::Event& event_; |
| 257 | }; |
| 258 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 259 | ReusedTask task(&call_count, reply_queue.get(), &event); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 260 | post_queue->PostTask(std::move(task)); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 261 | EXPECT_TRUE(event.Wait(TimeDelta::Seconds(1))); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 262 | } |
| 263 | |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 264 | TEST_P(TaskQueueTest, PostALot) { |
Artem Titov | 0e61fdd | 2021-07-25 19:50:14 | [diff] [blame] | 265 | // Waits until DecrementCount called `count` times. Thread safe. |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 266 | class BlockingCounter { |
| 267 | public: |
| 268 | explicit BlockingCounter(int initial_count) : count_(initial_count) {} |
| 269 | |
| 270 | void DecrementCount() { |
Harald Alvestrand | 78f905e | 2023-11-02 14:09:26 | [diff] [blame] | 271 | if (count_.DecRef() == webrtc::RefCountReleaseStatus::kDroppedLastRef) { |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 272 | event_.Set(); |
| 273 | } |
| 274 | } |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 275 | bool Wait(TimeDelta give_up_after) { return event_.Wait(give_up_after); } |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 276 | |
| 277 | private: |
| 278 | webrtc_impl::RefCounter count_; |
| 279 | rtc::Event event_; |
| 280 | }; |
| 281 | |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 282 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 283 | static constexpr int kTaskCount = 0xffff; |
| 284 | rtc::Event posting_done; |
| 285 | BlockingCounter all_destroyed(kTaskCount); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 286 | |
| 287 | int tasks_executed = 0; |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 288 | auto task_queue = CreateTaskQueue(factory, "PostALot"); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 289 | |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 290 | task_queue->PostTask([&] { |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 291 | // Post tasks from the queue to guarantee that the 1st task won't be |
| 292 | // executed before the last one is posted. |
| 293 | for (int i = 0; i < kTaskCount; ++i) { |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 294 | absl::Cleanup cleanup = [&] { all_destroyed.DecrementCount(); }; |
| 295 | task_queue->PostTask([&tasks_executed, cleanup = std::move(cleanup)] { |
| 296 | ++tasks_executed; |
| 297 | }); |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 298 | } |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 299 | |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 300 | posting_done.Set(); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 301 | }); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 302 | |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 303 | // Before destroying the task queue wait until all child tasks are posted. |
Danil Chapovalov | 95e0a60 | 2019-06-11 11:49:20 | [diff] [blame] | 304 | posting_done.Wait(rtc::Event::kForever); |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 305 | // Destroy the task queue. |
| 306 | task_queue = nullptr; |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 307 | |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 308 | // Expect all tasks are destroyed eventually. In some task queue |
| 309 | // implementations that might happen on a different thread after task queue is |
| 310 | // destroyed. |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 311 | EXPECT_TRUE(all_destroyed.Wait(TimeDelta::Minutes(1))); |
Danil Chapovalov | f504dd3 | 2019-06-05 12:16:59 | [diff] [blame] | 312 | EXPECT_LE(tasks_executed, kTaskCount); |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 313 | } |
| 314 | |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 315 | // Test posting two tasks that have shared state not protected by a |
| 316 | // lock. The TaskQueue should guarantee memory read-write order and |
| 317 | // FIFO task execution order, so the second task should always see the |
| 318 | // changes that were made by the first task. |
| 319 | // |
| 320 | // If the TaskQueue doesn't properly synchronize the execution of |
| 321 | // tasks, there will be a data race, which is undefined behavior. The |
| 322 | // EXPECT calls may randomly catch this, but to make the most of this |
| 323 | // unit test, run it under TSan or some other tool that is able to |
| 324 | // directly detect data races. |
| 325 | TEST_P(TaskQueueTest, PostTwoWithSharedUnprotectedState) { |
Ali Tofigh | 4b68194 | 2022-08-23 10:57:16 | [diff] [blame] | 326 | std::unique_ptr<webrtc::TaskQueueFactory> factory = GetParam()(nullptr); |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 327 | struct SharedState { |
| 328 | // First task will set this value to 1 and second will assert it. |
| 329 | int state = 0; |
| 330 | } state; |
| 331 | |
Danil Chapovalov | 710f3d3 | 2019-02-06 15:00:44 | [diff] [blame] | 332 | auto queue = CreateTaskQueue(factory, "PostTwoWithSharedUnprotectedState"); |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 333 | rtc::Event done; |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 334 | queue->PostTask([&state, &queue, &done] { |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 335 | // Post tasks from queue to guarantee, that 1st task won't be |
| 336 | // executed before the second one will be posted. |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 337 | queue->PostTask([&state] { state.state = 1; }); |
| 338 | queue->PostTask([&state, &done] { |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 339 | EXPECT_EQ(state.state, 1); |
| 340 | done.Set(); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 341 | }); |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 342 | // Check, that state changing tasks didn't start yet. |
| 343 | EXPECT_EQ(state.state, 0); |
Danil Chapovalov | 8feb6fd | 2022-07-05 09:01:27 | [diff] [blame] | 344 | }); |
Markus Handell | 2cfc1af | 2022-08-19 08:16:48 | [diff] [blame] | 345 | EXPECT_TRUE(done.Wait(TimeDelta::Seconds(1))); |
Artem Titov | 01f64e0 | 2019-01-31 12:31:09 | [diff] [blame] | 346 | } |
| 347 | |
Danil Chapovalov | 5ad16a5 | 2020-06-09 12:44:31 | [diff] [blame] | 348 | // TaskQueueTest is a set of tests for any implementation of the TaskQueueBase. |
| 349 | // Tests are instantiated next to the concrete implementation(s). |
| 350 | // https://github.com/google/googletest/blob/master/googletest/docs/advanced.md#creating-value-parameterized-abstract-tests |
| 351 | GTEST_ALLOW_UNINSTANTIATED_PARAMETERIZED_TEST(TaskQueueTest); |
| 352 | |
Danil Chapovalov | 33b716f | 2019-01-22 17:15:37 | [diff] [blame] | 353 | } // namespace |
| 354 | } // namespace webrtc |