commit 6f148566dc550c8752f62f8f771bf967bffa75bc
author philipel <philipel@webrtc.org> Thu Jul 30 10:04:59 2020
committer Commit Bot <commit-bot@chromium.org> Thu Jul 30 11:15:56 2020
tree c22a525c3e4937ab2115264713b3af1e89f849d7
parent c6cf9020349590026f28fa85d9ee92f0ff6e371c

Removed FrameBuffer::Start function.

Bug: webrtc:9106
Change-Id: I98cbc6d89b01e7c49b0595da5d5e446652418897
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/180581
Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org>
Commit-Queue: Philip Eliasson <philipel@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#31809}

modules/video_coding/frame_buffer2.cc

diff --git a/modules/video_coding/frame_buffer2.cc b/modules/video_coding/frame_buffer2.cc
index 88ac09c..fd65d7e 100644
--- a/modules/video_coding/frame_buffer2.cc
+++ b/modules/video_coding/frame_buffer2.cc
@@ -333,12 +333,6 @@
   protection_mode_ = mode;
 }
 
-void FrameBuffer::Start() {
-  TRACE_EVENT0("webrtc", "FrameBuffer::Start");
-  MutexLock lock(&mutex_);
-  stopped_ = false;
-}
-
 void FrameBuffer::Stop() {
   TRACE_EVENT0("webrtc", "FrameBuffer::Stop");
   MutexLock lock(&mutex_);

modules/video_coding/frame_buffer2.h

diff --git a/modules/video_coding/frame_buffer2.h b/modules/video_coding/frame_buffer2.h
index 7909000..c88ae89 100644
--- a/modules/video_coding/frame_buffer2.h
+++ b/modules/video_coding/frame_buffer2.h
@@ -71,10 +71,6 @@
   //                 implemented.
   void SetProtectionMode(VCMVideoProtection mode);
 
-  // Start the frame buffer, has no effect if the frame buffer is started.
-  // The frame buffer is started upon construction.
-  void Start();
-
   // Stop the frame buffer, causing any sleeping thread in NextFrame to
   // return immediately.
   void Stop();

video/video_receive_stream.cc

diff --git a/video/video_receive_stream.cc b/video/video_receive_stream.cc
index b4c6ddf..54b313c 100644
--- a/video/video_receive_stream.cc
+++ b/video/video_receive_stream.cc
@@ -319,8 +319,6 @@
   const bool protected_by_fec = config_.rtp.protected_by_flexfec ||
                                 rtp_video_stream_receiver_.IsUlpfecEnabled();
 
-  frame_buffer_->Start();
-
   if (rtp_video_stream_receiver_.IsRetransmissionsEnabled() &&
       protected_by_fec) {
     frame_buffer_->SetProtectionMode(kProtectionNackFEC);

video/video_receive_stream2.cc

diff --git a/video/video_receive_stream2.cc b/video/video_receive_stream2.cc
index 9413b72..e7961e2 100644
--- a/video/video_receive_stream2.cc
+++ b/video/video_receive_stream2.cc
@@ -295,8 +295,6 @@
   const bool protected_by_fec = config_.rtp.protected_by_flexfec ||
                                 rtp_video_stream_receiver_.IsUlpfecEnabled();
 
-  frame_buffer_->Start();
-
   if (rtp_video_stream_receiver_.IsRetransmissionsEnabled() &&
       protected_by_fec) {
     frame_buffer_->SetProtectionMode(kProtectionNackFEC);