From d9755eea22fd60a58bfa07dc616a12ffd43b5de2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niels=20M=C3=B6ller?= Date: Wed, 9 Oct 2019 10:59:03 +0200 Subject: [PATCH] Delete large up-front allocation in LibvpxVp8Encoder::InitEncode No longer useful after cl https://webrtc-review.googlesource.com/c/src/+/155163 Bug: chromium:1012256,webrtc:9378 Change-Id: I2ee000b72add0b34933b7954ad7c8bf0d69fc88e Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/156163 Reviewed-by: Philip Eliasson Commit-Queue: Niels Moller Cr-Commit-Position: refs/heads/master@{#29416} --- modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc | 6 ------ 1 file changed, 6 deletions(-) diff --git a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc index ee5fd436b2..556ac60479 100644 --- a/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc +++ b/modules/video_coding/codecs/vp8/libvpx_vp8_encoder.cc @@ -27,7 +27,6 @@ #include "api/video/video_timing.h" #include "api/video_codecs/vp8_temporal_layers.h" #include "api/video_codecs/vp8_temporal_layers_factory.h" -#include "common_video/libyuv/include/webrtc_libyuv.h" #include "modules/video_coding/codecs/interface/common_constants.h" #include "modules/video_coding/codecs/vp8/include/vp8.h" #include "modules/video_coding/include/video_error_codes.h" @@ -537,11 +536,6 @@ int LibvpxVp8Encoder::InitEncode(const VideoCodec* inst, downsampling_factors_[number_of_streams - 1].den = 1; } for (int i = 0; i < number_of_streams; ++i) { - // allocate memory for encoded image - size_t frame_capacity = - CalcBufferSize(VideoType::kI420, codec_.width, codec_.height); - encoded_images_[i].SetEncodedData( - EncodedImageBuffer::Create(frame_capacity)); encoded_images_[i]._completeFrame = true; } // populate encoder configuration with default values