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

Side by Side Diff: webrtc/modules/video_coding/codecs/test/videoprocessor_integrationtest.h

Issue 2946263002: Make VideoProcessor::Init void (always returning true). (Closed)
Patch Set: fix warning Created 3 years, 6 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
OLDNEW
1 /* 1 /*
2 * Copyright (c) 2012 The WebRTC project authors. All Rights Reserved. 2 * Copyright (c) 2012 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 331 matching lines...) Expand 10 before | Expand all | Expand 10 after
342 } 342 }
343 } 343 }
344 344
345 packet_manipulator_.reset(new test::PacketManipulatorImpl( 345 packet_manipulator_.reset(new test::PacketManipulatorImpl(
346 &packet_reader_, config_.networking_config, config_.verbose)); 346 &packet_reader_, config_.networking_config, config_.verbose));
347 processor_.reset(new test::VideoProcessorImpl( 347 processor_.reset(new test::VideoProcessorImpl(
348 encoder_.get(), decoder_.get(), analysis_frame_reader_.get(), 348 encoder_.get(), decoder_.get(), analysis_frame_reader_.get(),
349 analysis_frame_writer_.get(), packet_manipulator_.get(), config_, 349 analysis_frame_writer_.get(), packet_manipulator_.get(), config_,
350 &stats_, source_frame_writer_.get(), encoded_frame_writer_.get(), 350 &stats_, source_frame_writer_.get(), encoded_frame_writer_.get(),
351 decoded_frame_writer_.get())); 351 decoded_frame_writer_.get()));
352 RTC_CHECK(processor_->Init()); 352 processor_->Init();
353 } 353 }
354 354
355 // Reset quantities after each encoder update, update the target 355 // Reset quantities after each encoder update, update the target
356 // per-frame bandwidth. 356 // per-frame bandwidth.
357 void ResetRateControlMetrics(int num_frames_to_hit_target) { 357 void ResetRateControlMetrics(int num_frames_to_hit_target) {
358 for (int i = 0; i < num_temporal_layers_; i++) { 358 for (int i = 0; i < num_temporal_layers_; i++) {
359 num_frames_per_update_[i] = 0; 359 num_frames_per_update_[i] = 0;
360 sum_frame_size_mismatch_[i] = 0.0f; 360 sum_frame_size_mismatch_[i] = 0.0f;
361 sum_encoded_frame_size_[i] = 0.0f; 361 sum_encoded_frame_size_[i] = 0.0f;
362 encoding_bitrate_[i] = 0.0f; 362 encoding_bitrate_[i] = 0.0f;
(...skipping 471 matching lines...) Expand 10 before | Expand all | Expand 10 after
834 834
835 // Codec and network settings. 835 // Codec and network settings.
836 float packet_loss_probability_; 836 float packet_loss_probability_;
837 int num_temporal_layers_; 837 int num_temporal_layers_;
838 }; 838 };
839 839
840 } // namespace test 840 } // namespace test
841 } // namespace webrtc 841 } // namespace webrtc
842 842
843 #endif // WEBRTC_MODULES_VIDEO_CODING_CODECS_TEST_VIDEOPROCESSOR_INTEGRATIONTES T_H_ 843 #endif // WEBRTC_MODULES_VIDEO_CODING_CODECS_TEST_VIDEOPROCESSOR_INTEGRATIONTES T_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698