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

Side by Side Diff: webrtc/video/vie_encoder_unittest.cc

Issue 2359153004: Fix vie_encoder_unittest.cc. (Closed)
Patch Set: Created 4 years, 3 months 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 unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2016 The WebRTC project authors. All Rights Reserved.
3 * 3 *
4 * Use of this source code is governed by a BSD-style license 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 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 6 * tree. An additional intellectual property rights grant can be found
7 * in the file PATENTS. All contributing project authors may 7 * in the file PATENTS. All contributing project authors may
8 * be found in the AUTHORS file in the root of the source tree. 8 * be found in the AUTHORS file in the root of the source tree.
9 */ 9 */
10 10
(...skipping 264 matching lines...) Expand 10 before | Expand all | Expand 10 after
275 sink_.WaitForEncodedFrame(3); 275 sink_.WaitForEncodedFrame(3);
276 276
277 vie_encoder_->Stop(); 277 vie_encoder_->Stop();
278 } 278 }
279 279
280 TEST_F(ViEEncoderTest, ConfigureEncoderTriggersOnEncoderConfigurationChanged) { 280 TEST_F(ViEEncoderTest, ConfigureEncoderTriggersOnEncoderConfigurationChanged) {
281 const int kTargetBitrateBps = 100000; 281 const int kTargetBitrateBps = 100000;
282 vie_encoder_->OnBitrateUpdated(kTargetBitrateBps, 0, 0); 282 vie_encoder_->OnBitrateUpdated(kTargetBitrateBps, 0, 0);
283 283
284 // Capture a frame and wait for it to synchronize with the encoder thread. 284 // Capture a frame and wait for it to synchronize with the encoder thread.
285 vie_encoder_->IncomingCapturedFrame(CreateFrame(1, nullptr)); 285 video_source_.IncomingCapturedFrame(CreateFrame(1, nullptr));
286 sink_.WaitForEncodedFrame(1); 286 sink_.WaitForEncodedFrame(1);
287 EXPECT_EQ(1, sink_.number_of_reconfigurations()); 287 EXPECT_EQ(1, sink_.number_of_reconfigurations());
288 288
289 VideoEncoderConfig video_encoder_config; 289 VideoEncoderConfig video_encoder_config;
290 video_encoder_config.streams = test::CreateVideoStreams(1); 290 video_encoder_config.streams = test::CreateVideoStreams(1);
291 video_encoder_config.min_transmit_bitrate_bps = 9999; 291 video_encoder_config.min_transmit_bitrate_bps = 9999;
292 vie_encoder_->ConfigureEncoder(std::move(video_encoder_config), 1440); 292 vie_encoder_->ConfigureEncoder(std::move(video_encoder_config), 1440);
293 293
294 // Capture a frame and wait for it to synchronize with the encoder thread. 294 // Capture a frame and wait for it to synchronize with the encoder thread.
295 vie_encoder_->IncomingCapturedFrame(CreateFrame(2, nullptr)); 295 video_source_.IncomingCapturedFrame(CreateFrame(2, nullptr));
296 sink_.WaitForEncodedFrame(2); 296 sink_.WaitForEncodedFrame(2);
297 EXPECT_EQ(2, sink_.number_of_reconfigurations()); 297 EXPECT_EQ(2, sink_.number_of_reconfigurations());
298 EXPECT_EQ(9999, sink_.last_min_transmit_bitrate()); 298 EXPECT_EQ(9999, sink_.last_min_transmit_bitrate());
299 299
300 vie_encoder_->Stop(); 300 vie_encoder_->Stop();
301 } 301 }
302 302
303 } // namespace webrtc 303 } // namespace webrtc
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698