perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 1 | /* |
| 2 | * Copyright 2016 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 | |
Jonas Olsson | a4d8737 | 2019-07-05 17:08:33 | [diff] [blame] | 11 | #include "media/base/video_broadcaster.h" |
| 12 | |
Steve Anton | e78bcb9 | 2017-10-31 16:53:08 | [diff] [blame] | 13 | #include <limits> |
| 14 | |
Yves Gerey | 3e70781 | 2018-11-28 15:47:49 | [diff] [blame] | 15 | #include "absl/types/optional.h" |
Mirko Bonadei | 92ea95e | 2017-09-15 04:47:31 | [diff] [blame] | 16 | #include "api/video/i420_buffer.h" |
| 17 | #include "api/video/video_frame.h" |
Yves Gerey | 3e70781 | 2018-11-28 15:47:49 | [diff] [blame] | 18 | #include "api/video/video_rotation.h" |
Markus Handell | 6fa9e68 | 2021-10-13 20:50:53 | [diff] [blame] | 19 | #include "api/video/video_source_interface.h" |
Steve Anton | 10542f2 | 2019-01-11 17:11:00 | [diff] [blame] | 20 | #include "media/base/fake_video_renderer.h" |
Markus Handell | 6fa9e68 | 2021-10-13 20:50:53 | [diff] [blame] | 21 | #include "test/gmock.h" |
Yves Gerey | 3e70781 | 2018-11-28 15:47:49 | [diff] [blame] | 22 | #include "test/gtest.h" |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 23 | |
Jonas Olsson | a4d8737 | 2019-07-05 17:08:33 | [diff] [blame] | 24 | using cricket::FakeVideoRenderer; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 25 | using rtc::VideoBroadcaster; |
| 26 | using rtc::VideoSinkWants; |
Jonas Oreland | 0deda15 | 2022-09-23 10:08:57 | [diff] [blame] | 27 | using FrameSize = rtc::VideoSinkWants::FrameSize; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 28 | |
Markus Handell | 6fa9e68 | 2021-10-13 20:50:53 | [diff] [blame] | 29 | using ::testing::AllOf; |
| 30 | using ::testing::Eq; |
| 31 | using ::testing::Field; |
| 32 | using ::testing::Mock; |
| 33 | using ::testing::Optional; |
| 34 | |
| 35 | class MockSink : public rtc::VideoSinkInterface<webrtc::VideoFrame> { |
| 36 | public: |
| 37 | void OnFrame(const webrtc::VideoFrame&) override {} |
| 38 | |
| 39 | MOCK_METHOD(void, |
| 40 | OnConstraintsChanged, |
| 41 | (const webrtc::VideoTrackSourceConstraints& constraints), |
| 42 | (override)); |
| 43 | }; |
| 44 | |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 45 | TEST(VideoBroadcasterTest, frame_wanted) { |
| 46 | VideoBroadcaster broadcaster; |
| 47 | EXPECT_FALSE(broadcaster.frame_wanted()); |
| 48 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 49 | FakeVideoRenderer sink; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 50 | broadcaster.AddOrUpdateSink(&sink, rtc::VideoSinkWants()); |
| 51 | EXPECT_TRUE(broadcaster.frame_wanted()); |
| 52 | |
| 53 | broadcaster.RemoveSink(&sink); |
| 54 | EXPECT_FALSE(broadcaster.frame_wanted()); |
| 55 | } |
| 56 | |
| 57 | TEST(VideoBroadcasterTest, OnFrame) { |
| 58 | VideoBroadcaster broadcaster; |
| 59 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 60 | FakeVideoRenderer sink1; |
| 61 | FakeVideoRenderer sink2; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 62 | broadcaster.AddOrUpdateSink(&sink1, rtc::VideoSinkWants()); |
| 63 | broadcaster.AddOrUpdateSink(&sink2, rtc::VideoSinkWants()); |
nisse | df2ceb8 | 2016-12-15 14:29:53 | [diff] [blame] | 64 | static int kWidth = 100; |
| 65 | static int kHeight = 50; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 66 | |
nisse | df2ceb8 | 2016-12-15 14:29:53 | [diff] [blame] | 67 | rtc::scoped_refptr<webrtc::I420Buffer> buffer( |
| 68 | webrtc::I420Buffer::Create(kWidth, kHeight)); |
| 69 | // Initialize, to avoid warnings on use of initialized values. |
Niels Möller | ba2de58 | 2022-04-20 14:46:26 | [diff] [blame] | 70 | webrtc::I420Buffer::SetBlack(buffer.get()); |
nisse | df2ceb8 | 2016-12-15 14:29:53 | [diff] [blame] | 71 | |
Artem Titov | 1ebfb6a | 2019-01-03 22:49:37 | [diff] [blame] | 72 | webrtc::VideoFrame frame = webrtc::VideoFrame::Builder() |
| 73 | .set_video_frame_buffer(buffer) |
| 74 | .set_rotation(webrtc::kVideoRotation_0) |
| 75 | .set_timestamp_us(0) |
| 76 | .build(); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 77 | |
| 78 | broadcaster.OnFrame(frame); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 79 | EXPECT_EQ(1, sink1.num_rendered_frames()); |
| 80 | EXPECT_EQ(1, sink2.num_rendered_frames()); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 81 | |
| 82 | broadcaster.RemoveSink(&sink1); |
| 83 | broadcaster.OnFrame(frame); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 84 | EXPECT_EQ(1, sink1.num_rendered_frames()); |
| 85 | EXPECT_EQ(2, sink2.num_rendered_frames()); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 86 | |
| 87 | broadcaster.AddOrUpdateSink(&sink1, rtc::VideoSinkWants()); |
| 88 | broadcaster.OnFrame(frame); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 89 | EXPECT_EQ(2, sink1.num_rendered_frames()); |
| 90 | EXPECT_EQ(3, sink2.num_rendered_frames()); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 91 | } |
| 92 | |
| 93 | TEST(VideoBroadcasterTest, AppliesRotationIfAnySinkWantsRotationApplied) { |
| 94 | VideoBroadcaster broadcaster; |
deadbeef | f5629ad | 2016-03-18 18:38:26 | [diff] [blame] | 95 | EXPECT_FALSE(broadcaster.wants().rotation_applied); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 96 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 97 | FakeVideoRenderer sink1; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 98 | VideoSinkWants wants1; |
| 99 | wants1.rotation_applied = false; |
| 100 | |
| 101 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 102 | EXPECT_FALSE(broadcaster.wants().rotation_applied); |
| 103 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 104 | FakeVideoRenderer sink2; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 105 | VideoSinkWants wants2; |
| 106 | wants2.rotation_applied = true; |
| 107 | |
| 108 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 109 | EXPECT_TRUE(broadcaster.wants().rotation_applied); |
| 110 | |
| 111 | broadcaster.RemoveSink(&sink2); |
| 112 | EXPECT_FALSE(broadcaster.wants().rotation_applied); |
| 113 | } |
| 114 | |
| 115 | TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxPixelCount) { |
| 116 | VideoBroadcaster broadcaster; |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 117 | EXPECT_EQ(std::numeric_limits<int>::max(), |
| 118 | broadcaster.wants().max_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 119 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 120 | FakeVideoRenderer sink1; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 121 | VideoSinkWants wants1; |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 122 | wants1.max_pixel_count = 1280 * 720; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 123 | |
| 124 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 125 | EXPECT_EQ(1280 * 720, broadcaster.wants().max_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 126 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 127 | FakeVideoRenderer sink2; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 128 | VideoSinkWants wants2; |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 129 | wants2.max_pixel_count = 640 * 360; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 130 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 131 | EXPECT_EQ(640 * 360, broadcaster.wants().max_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 132 | |
| 133 | broadcaster.RemoveSink(&sink2); |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 134 | EXPECT_EQ(1280 * 720, broadcaster.wants().max_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 135 | } |
| 136 | |
sprang | 84a3759 | 2017-02-10 15:04:27 | [diff] [blame] | 137 | TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxAndTargetPixelCount) { |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 138 | VideoBroadcaster broadcaster; |
sprang | 84a3759 | 2017-02-10 15:04:27 | [diff] [blame] | 139 | EXPECT_TRUE(!broadcaster.wants().target_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 140 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 141 | FakeVideoRenderer sink1; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 142 | VideoSinkWants wants1; |
Oskar Sundbom | 7880758 | 2017-11-16 10:09:55 | [diff] [blame] | 143 | wants1.target_pixel_count = 1280 * 720; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 144 | |
| 145 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
sprang | 84a3759 | 2017-02-10 15:04:27 | [diff] [blame] | 146 | EXPECT_EQ(1280 * 720, *broadcaster.wants().target_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 147 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 148 | FakeVideoRenderer sink2; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 149 | VideoSinkWants wants2; |
Oskar Sundbom | 7880758 | 2017-11-16 10:09:55 | [diff] [blame] | 150 | wants2.target_pixel_count = 640 * 360; |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 151 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
sprang | 84a3759 | 2017-02-10 15:04:27 | [diff] [blame] | 152 | EXPECT_EQ(640 * 360, *broadcaster.wants().target_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 153 | |
| 154 | broadcaster.RemoveSink(&sink2); |
sprang | 84a3759 | 2017-02-10 15:04:27 | [diff] [blame] | 155 | EXPECT_EQ(1280 * 720, *broadcaster.wants().target_pixel_count); |
perkj | 2d5f091 | 2016-02-29 08:04:41 | [diff] [blame] | 156 | } |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 157 | |
sprang | c5d62e2 | 2017-04-03 06:53:04 | [diff] [blame] | 158 | TEST(VideoBroadcasterTest, AppliesMinOfSinkWantsMaxFramerate) { |
| 159 | VideoBroadcaster broadcaster; |
| 160 | EXPECT_EQ(std::numeric_limits<int>::max(), |
| 161 | broadcaster.wants().max_framerate_fps); |
| 162 | |
| 163 | FakeVideoRenderer sink1; |
| 164 | VideoSinkWants wants1; |
| 165 | wants1.max_framerate_fps = 30; |
| 166 | |
| 167 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 168 | EXPECT_EQ(30, broadcaster.wants().max_framerate_fps); |
| 169 | |
| 170 | FakeVideoRenderer sink2; |
| 171 | VideoSinkWants wants2; |
| 172 | wants2.max_framerate_fps = 15; |
| 173 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 174 | EXPECT_EQ(15, broadcaster.wants().max_framerate_fps); |
| 175 | |
| 176 | broadcaster.RemoveSink(&sink2); |
| 177 | EXPECT_EQ(30, broadcaster.wants().max_framerate_fps); |
| 178 | } |
| 179 | |
Rasmus Brandt | 5cad55b | 2019-12-19 08:47:11 | [diff] [blame] | 180 | TEST(VideoBroadcasterTest, |
| 181 | AppliesLeastCommonMultipleOfSinkWantsResolutionAlignment) { |
| 182 | VideoBroadcaster broadcaster; |
| 183 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 1); |
| 184 | |
| 185 | FakeVideoRenderer sink1; |
| 186 | VideoSinkWants wants1; |
| 187 | wants1.resolution_alignment = 2; |
| 188 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 189 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 2); |
| 190 | |
| 191 | FakeVideoRenderer sink2; |
| 192 | VideoSinkWants wants2; |
| 193 | wants2.resolution_alignment = 3; |
| 194 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 195 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 6); |
| 196 | |
| 197 | FakeVideoRenderer sink3; |
| 198 | VideoSinkWants wants3; |
| 199 | wants3.resolution_alignment = 4; |
| 200 | broadcaster.AddOrUpdateSink(&sink3, wants3); |
| 201 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 12); |
| 202 | |
| 203 | broadcaster.RemoveSink(&sink2); |
| 204 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 4); |
| 205 | } |
| 206 | |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 207 | TEST(VideoBroadcasterTest, SinkWantsBlackFrames) { |
| 208 | VideoBroadcaster broadcaster; |
| 209 | EXPECT_TRUE(!broadcaster.wants().black_frames); |
| 210 | |
| 211 | FakeVideoRenderer sink1; |
| 212 | VideoSinkWants wants1; |
| 213 | wants1.black_frames = true; |
| 214 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 215 | |
| 216 | FakeVideoRenderer sink2; |
| 217 | VideoSinkWants wants2; |
nisse | efec590 | 2016-06-09 07:31:39 | [diff] [blame] | 218 | wants2.black_frames = false; |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 219 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 220 | |
nisse | efec590 | 2016-06-09 07:31:39 | [diff] [blame] | 221 | rtc::scoped_refptr<webrtc::I420Buffer> buffer( |
nisse | af91689 | 2017-01-10 15:44:26 | [diff] [blame] | 222 | webrtc::I420Buffer::Create(100, 200)); |
nisse | efec590 | 2016-06-09 07:31:39 | [diff] [blame] | 223 | // Makes it not all black. |
| 224 | buffer->InitializeData(); |
| 225 | |
Artem Titov | 1ebfb6a | 2019-01-03 22:49:37 | [diff] [blame] | 226 | webrtc::VideoFrame frame1 = webrtc::VideoFrame::Builder() |
| 227 | .set_video_frame_buffer(buffer) |
| 228 | .set_rotation(webrtc::kVideoRotation_0) |
| 229 | .set_timestamp_us(10) |
| 230 | .build(); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 231 | broadcaster.OnFrame(frame1); |
| 232 | EXPECT_TRUE(sink1.black_frame()); |
nisse | 306d52b | 2016-09-06 14:52:40 | [diff] [blame] | 233 | EXPECT_EQ(10, sink1.timestamp_us()); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 234 | EXPECT_FALSE(sink2.black_frame()); |
nisse | 306d52b | 2016-09-06 14:52:40 | [diff] [blame] | 235 | EXPECT_EQ(10, sink2.timestamp_us()); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 236 | |
| 237 | // Switch the sink wants. |
| 238 | wants1.black_frames = false; |
| 239 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 240 | wants2.black_frames = true; |
| 241 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 242 | |
Artem Titov | 1ebfb6a | 2019-01-03 22:49:37 | [diff] [blame] | 243 | webrtc::VideoFrame frame2 = webrtc::VideoFrame::Builder() |
| 244 | .set_video_frame_buffer(buffer) |
| 245 | .set_rotation(webrtc::kVideoRotation_0) |
| 246 | .set_timestamp_us(30) |
| 247 | .build(); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 248 | broadcaster.OnFrame(frame2); |
| 249 | EXPECT_FALSE(sink1.black_frame()); |
nisse | 306d52b | 2016-09-06 14:52:40 | [diff] [blame] | 250 | EXPECT_EQ(30, sink1.timestamp_us()); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 251 | EXPECT_TRUE(sink2.black_frame()); |
nisse | 306d52b | 2016-09-06 14:52:40 | [diff] [blame] | 252 | EXPECT_EQ(30, sink2.timestamp_us()); |
perkj | d6c3954 | 2016-03-17 09:35:23 | [diff] [blame] | 253 | } |
Markus Handell | 6fa9e68 | 2021-10-13 20:50:53 | [diff] [blame] | 254 | |
| 255 | TEST(VideoBroadcasterTest, ConstraintsChangedNotCalledOnSinkAddition) { |
| 256 | MockSink sink; |
| 257 | VideoBroadcaster broadcaster; |
| 258 | EXPECT_CALL(sink, OnConstraintsChanged).Times(0); |
| 259 | broadcaster.AddOrUpdateSink(&sink, VideoSinkWants()); |
| 260 | } |
| 261 | |
| 262 | TEST(VideoBroadcasterTest, ForwardsLastConstraintsOnAdd) { |
| 263 | MockSink sink; |
| 264 | VideoBroadcaster broadcaster; |
| 265 | broadcaster.ProcessConstraints(webrtc::VideoTrackSourceConstraints{2, 3}); |
| 266 | broadcaster.ProcessConstraints(webrtc::VideoTrackSourceConstraints{1, 4}); |
| 267 | EXPECT_CALL( |
| 268 | sink, |
| 269 | OnConstraintsChanged(AllOf( |
| 270 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, Optional(1)), |
| 271 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, Optional(4))))); |
| 272 | broadcaster.AddOrUpdateSink(&sink, VideoSinkWants()); |
| 273 | } |
| 274 | |
| 275 | TEST(VideoBroadcasterTest, UpdatesOnlyNewSinksWithConstraints) { |
| 276 | MockSink sink1; |
| 277 | VideoBroadcaster broadcaster; |
| 278 | broadcaster.AddOrUpdateSink(&sink1, VideoSinkWants()); |
| 279 | broadcaster.ProcessConstraints(webrtc::VideoTrackSourceConstraints{1, 4}); |
| 280 | Mock::VerifyAndClearExpectations(&sink1); |
| 281 | EXPECT_CALL(sink1, OnConstraintsChanged).Times(0); |
| 282 | MockSink sink2; |
| 283 | EXPECT_CALL( |
| 284 | sink2, |
| 285 | OnConstraintsChanged(AllOf( |
| 286 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, Optional(1)), |
| 287 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, Optional(4))))); |
| 288 | broadcaster.AddOrUpdateSink(&sink2, VideoSinkWants()); |
| 289 | } |
| 290 | |
| 291 | TEST(VideoBroadcasterTest, ForwardsConstraintsToSink) { |
| 292 | MockSink sink; |
| 293 | VideoBroadcaster broadcaster; |
| 294 | EXPECT_CALL(sink, OnConstraintsChanged).Times(0); |
| 295 | broadcaster.AddOrUpdateSink(&sink, VideoSinkWants()); |
| 296 | Mock::VerifyAndClearExpectations(&sink); |
| 297 | |
| 298 | EXPECT_CALL(sink, OnConstraintsChanged(AllOf( |
| 299 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, |
| 300 | Eq(absl::nullopt)), |
| 301 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, |
| 302 | Eq(absl::nullopt))))); |
| 303 | broadcaster.ProcessConstraints( |
| 304 | webrtc::VideoTrackSourceConstraints{absl::nullopt, absl::nullopt}); |
| 305 | Mock::VerifyAndClearExpectations(&sink); |
| 306 | |
| 307 | EXPECT_CALL( |
| 308 | sink, |
| 309 | OnConstraintsChanged(AllOf( |
| 310 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, |
| 311 | Eq(absl::nullopt)), |
| 312 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, Optional(3))))); |
| 313 | broadcaster.ProcessConstraints( |
| 314 | webrtc::VideoTrackSourceConstraints{absl::nullopt, 3}); |
| 315 | Mock::VerifyAndClearExpectations(&sink); |
| 316 | |
| 317 | EXPECT_CALL( |
| 318 | sink, |
| 319 | OnConstraintsChanged(AllOf( |
| 320 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, Optional(2)), |
| 321 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, |
| 322 | Eq(absl::nullopt))))); |
| 323 | broadcaster.ProcessConstraints( |
| 324 | webrtc::VideoTrackSourceConstraints{2, absl::nullopt}); |
| 325 | Mock::VerifyAndClearExpectations(&sink); |
| 326 | |
| 327 | EXPECT_CALL( |
| 328 | sink, |
| 329 | OnConstraintsChanged(AllOf( |
| 330 | Field(&webrtc::VideoTrackSourceConstraints::min_fps, Optional(2)), |
| 331 | Field(&webrtc::VideoTrackSourceConstraints::max_fps, Optional(3))))); |
| 332 | broadcaster.ProcessConstraints(webrtc::VideoTrackSourceConstraints{2, 3}); |
| 333 | } |
Jonas Oreland | 0deda15 | 2022-09-23 10:08:57 | [diff] [blame] | 334 | |
| 335 | TEST(VideoBroadcasterTest, AppliesMaxOfSinkWantsRequestedResolution) { |
| 336 | VideoBroadcaster broadcaster; |
| 337 | |
| 338 | FakeVideoRenderer sink1; |
| 339 | VideoSinkWants wants1; |
Jonas Oreland | 1d3452f | 2023-05-12 14:36:41 | [diff] [blame] | 340 | wants1.is_active = true; |
Jonas Oreland | 0deda15 | 2022-09-23 10:08:57 | [diff] [blame] | 341 | wants1.requested_resolution = FrameSize(640, 360); |
| 342 | |
| 343 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 344 | EXPECT_EQ(FrameSize(640, 360), *broadcaster.wants().requested_resolution); |
| 345 | |
| 346 | FakeVideoRenderer sink2; |
| 347 | VideoSinkWants wants2; |
Jonas Oreland | 1d3452f | 2023-05-12 14:36:41 | [diff] [blame] | 348 | wants2.is_active = true; |
Jonas Oreland | 0deda15 | 2022-09-23 10:08:57 | [diff] [blame] | 349 | wants2.requested_resolution = FrameSize(650, 350); |
| 350 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 351 | EXPECT_EQ(FrameSize(650, 360), *broadcaster.wants().requested_resolution); |
| 352 | |
| 353 | broadcaster.RemoveSink(&sink2); |
| 354 | EXPECT_EQ(FrameSize(640, 360), *broadcaster.wants().requested_resolution); |
| 355 | } |
| 356 | |
| 357 | TEST(VideoBroadcasterTest, AnyActive) { |
| 358 | VideoBroadcaster broadcaster; |
| 359 | |
| 360 | FakeVideoRenderer sink1; |
| 361 | VideoSinkWants wants1; |
| 362 | wants1.is_active = false; |
| 363 | |
| 364 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 365 | EXPECT_EQ(false, broadcaster.wants().is_active); |
| 366 | |
| 367 | FakeVideoRenderer sink2; |
| 368 | VideoSinkWants wants2; |
| 369 | wants2.is_active = true; |
| 370 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 371 | EXPECT_EQ(true, broadcaster.wants().is_active); |
| 372 | |
| 373 | broadcaster.RemoveSink(&sink2); |
| 374 | EXPECT_EQ(false, broadcaster.wants().is_active); |
| 375 | } |
| 376 | |
| 377 | TEST(VideoBroadcasterTest, AnyActiveWithoutRequestedResolution) { |
| 378 | VideoBroadcaster broadcaster; |
| 379 | |
| 380 | FakeVideoRenderer sink1; |
| 381 | VideoSinkWants wants1; |
| 382 | wants1.is_active = true; |
| 383 | wants1.requested_resolution = FrameSize(640, 360); |
| 384 | |
| 385 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 386 | EXPECT_EQ( |
| 387 | false, |
| 388 | broadcaster.wants().aggregates->any_active_without_requested_resolution); |
| 389 | |
| 390 | FakeVideoRenderer sink2; |
| 391 | VideoSinkWants wants2; |
| 392 | wants2.is_active = true; |
| 393 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 394 | EXPECT_EQ( |
| 395 | true, |
| 396 | broadcaster.wants().aggregates->any_active_without_requested_resolution); |
| 397 | |
| 398 | broadcaster.RemoveSink(&sink2); |
| 399 | EXPECT_EQ( |
| 400 | false, |
| 401 | broadcaster.wants().aggregates->any_active_without_requested_resolution); |
| 402 | } |
Jonas Oreland | 43f0f29 | 2022-10-07 13:37:17 | [diff] [blame] | 403 | |
| 404 | // This verifies that the VideoSinkWants from a Sink that is_active = false |
| 405 | // is ignored IF there is an active sink using new api (Requested_Resolution). |
| 406 | // The uses resolution_alignment for verification. |
| 407 | TEST(VideoBroadcasterTest, IgnoreInactiveSinkIfNewApiUsed) { |
| 408 | VideoBroadcaster broadcaster; |
| 409 | |
| 410 | FakeVideoRenderer sink1; |
| 411 | VideoSinkWants wants1; |
| 412 | wants1.is_active = true; |
| 413 | wants1.requested_resolution = FrameSize(640, 360); |
| 414 | wants1.resolution_alignment = 2; |
| 415 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 416 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 2); |
| 417 | |
| 418 | FakeVideoRenderer sink2; |
| 419 | VideoSinkWants wants2; |
| 420 | wants2.is_active = true; |
| 421 | wants2.resolution_alignment = 8; |
| 422 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 423 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 8); |
| 424 | |
| 425 | // Now wants2 will be ignored. |
| 426 | wants2.is_active = false; |
| 427 | broadcaster.AddOrUpdateSink(&sink2, wants2); |
| 428 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 2); |
| 429 | |
| 430 | // But when wants1 is inactive, wants2 matters again. |
| 431 | wants1.is_active = false; |
| 432 | broadcaster.AddOrUpdateSink(&sink1, wants1); |
| 433 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 8); |
| 434 | |
| 435 | // inactive wants1 (new api) is always ignored. |
| 436 | broadcaster.RemoveSink(&sink2); |
| 437 | EXPECT_EQ(broadcaster.wants().resolution_alignment, 1); |
| 438 | } |