Chromium Code Reviews| Index: webrtc/modules/video_coding/main/source/generic_encoder.cc |
| diff --git a/webrtc/modules/video_coding/main/source/generic_encoder.cc b/webrtc/modules/video_coding/main/source/generic_encoder.cc |
| index 31c3f1715f8d585b1141d182dde1e9f2c739faf9..b77a4a7aff56fe73b7e48770248b0fa51fa84ff1 100644 |
| --- a/webrtc/modules/video_coding/main/source/generic_encoder.cc |
| +++ b/webrtc/modules/video_coding/main/source/generic_encoder.cc |
| @@ -140,10 +140,6 @@ VCMGenericEncoder::InitEncode(const VideoCodec* settings, |
| int32_t VCMGenericEncoder::Encode(const VideoFrame& inputFrame, |
| const CodecSpecificInfo* codecSpecificInfo, |
| const std::vector<FrameType>& frameTypes) { |
|
stefan-webrtc
2015/10/01 09:21:20
DCHECK on frame types being valid video types?
pbos-webrtc
2015/10/06 15:42:45
Done.
|
| - std::vector<VideoFrameType> video_frame_types(frameTypes.size(), |
| - kDeltaFrame); |
| - VCMEncodedFrame::ConvertFrameTypes(frameTypes, &video_frame_types); |
| - |
| rotation_ = inputFrame.rotation(); |
| if (vcm_encoded_frame_callback_) { |
| @@ -153,12 +149,11 @@ int32_t VCMGenericEncoder::Encode(const VideoFrame& inputFrame, |
| vcm_encoded_frame_callback_->SetRotation(rotation_); |
| } |
| - int32_t result = |
| - encoder_->Encode(inputFrame, codecSpecificInfo, &video_frame_types); |
| + int32_t result = encoder_->Encode(inputFrame, codecSpecificInfo, &frameTypes); |
| if (is_screenshare_ && |
| result == WEBRTC_VIDEO_CODEC_TARGET_BITRATE_OVERSHOOT) { |
| // Target bitrate exceeded, encoder state has been reset - try again. |
| - return encoder_->Encode(inputFrame, codecSpecificInfo, &video_frame_types); |
| + return encoder_->Encode(inputFrame, codecSpecificInfo, &frameTypes); |
| } |
| return result; |
| @@ -223,10 +218,7 @@ VCMGenericEncoder::SetPeriodicKeyFrames(bool enable) |
| int32_t VCMGenericEncoder::RequestFrame( |
| const std::vector<FrameType>& frame_types) { |
| VideoFrame image; |
| - std::vector<VideoFrameType> video_frame_types(frame_types.size(), |
| - kDeltaFrame); |
| - VCMEncodedFrame::ConvertFrameTypes(frame_types, &video_frame_types); |
| - return encoder_->Encode(image, NULL, &video_frame_types); |
| + return encoder_->Encode(image, NULL, &frame_types); |
| } |
| int32_t |