From b1070353ad717763a6433061cc9a76b0438587f0 Mon Sep 17 00:00:00 2001
From: Elad Alon <eladalon@webrtc.org>
Date: Tue, 30 Apr 2019 12:35:02 +0200
Subject: [PATCH] Prevent memory over-consumption by DefaultTemporalLayers

Erase pending frames after they had been used.

Bug: chromium:956510
Change-Id: I55eec75dab9bbe350dd1b089953bdc2979d989fd
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/134641
Commit-Queue: Elad Alon <eladalon@webrtc.org>
Reviewed-by: Rasmus Brandt <brandtr@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#27811}
---
 modules/video_coding/codecs/vp8/default_temporal_layers.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/modules/video_coding/codecs/vp8/default_temporal_layers.cc b/modules/video_coding/codecs/vp8/default_temporal_layers.cc
index 5a434d9518..8fa8f0d774 100644
--- a/modules/video_coding/codecs/vp8/default_temporal_layers.cc
+++ b/modules/video_coding/codecs/vp8/default_temporal_layers.cc
@@ -547,6 +547,8 @@ void DefaultTemporalLayers::OnEncodeDone(size_t stream_index,
       }
     }
   }
+
+  pending_frames_.erase(pending_frame);
 }
 
 void DefaultTemporalLayers::OnFrameDropped(size_t stream_index,