zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use RC_TIMESTAMP_MODE for OpenH264.

Browse Source 
Performs rate control based on timestamp deltas instead of announced
frame rate.

BUG=webrtc:5855
R=hbos@webrtc.org

Review URL: https://codereview.webrtc.org/1945763002 .

Cr-Commit-Position: refs/heads/master@{#12611}
This commit is contained in:
Peter Boström
2016-05-03 13:59:59 -04:00
parent c8fe991a3d
commit c4deee49a3
1 changed files with 6 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
webrtc/modules/video_coding/codecs/h264/h264_encoder_impl.cc
View File

@ -224,7 +224,7 @@ int32_t H264EncoderImpl::InitEncode(const VideoCodec* codec_settings,
init_params.iTargetBitrate = codec_settings_.targetBitrate * 1000;
init_params.iMaxBitrate = codec_settings_.maxBitrate * 1000;
// Rate Control mode
init_params.iRCMode = RC_BITRATE_MODE;
init_params.iRCMode = RC_TIMESTAMP_MODE;
init_params.fMaxFrameRate = static_cast<float>(codec_settings_.maxFramerate);
// The following parameters are extension parameters (they're in SEncParamExt,
@ -343,21 +343,13 @@ int32_t H264EncoderImpl::Encode(
return WEBRTC_VIDEO_CODEC_ERR_SIZE;
}
bool force_key_frame = false;
if (frame_types != nullptr) {
// We only support a single stream.
RTC_DCHECK_EQ(frame_types->size(), static_cast<size_t>(1));
// Skip frame?
if ((*frame_types)[0] == kEmptyFrame) {
return WEBRTC_VIDEO_CODEC_OK;
}
// Force key frame?
force_key_frame = (*frame_types)[0] == kVideoFrameKey;
}
if (force_key_frame) {
RTC_DCHECK(frame_types != nullptr);
// We only support a single stream.
RTC_DCHECK_EQ(1u, frame_types->size());
// Force key frame?
if ((*frame_types)[0] == kVideoFrameKey) {
// API doc says ForceIntraFrame(false) does nothing, but calling this
// function forces a key frame regardless of the |bIDR| argument's value.
// (If every frame is a key frame we get lag/delays.)
openh264_encoder_->ForceIntraFrame(true);
}