Remove layer_sync from TL frame config.

webrtc/modules/video_coding/codecs/vp8/screenshare_layers.cc

 /* Copyright (c) 2013 The WebRTC project authors. All Rights Reserved.
 *
 *  Use of this source code is governed by a BSD-style license
 *  that can be found in the LICENSE file in the root of the source
 *  tree. An additional intellectual property rights grant can be found
 *  in the file PATENTS.  All contributing project authors may
 *  be found in the AUTHORS file in the root of the source tree.
 */
 
 #include "webrtc/modules/video_coding/codecs/vp8/screenshare_layers.h"
@@ skipping 75 matching lines @@
 
 uint8_t ScreenshareLayers::Tl0PicIdx() const {
   return tl0_pic_idx_;
 }
 
 TemporalLayers::FrameConfig ScreenshareLayers::UpdateLayerConfig(
     uint32_t timestamp) {
   if (number_of_temporal_layers_ <= 1) {
     // No flags needed for 1 layer screenshare.
     // TODO(pbos): Consider updating only last, and not all buffers.
-    return TemporalLayers::FrameConfig(kReferenceAndUpdate, kReferenceAndUpdate,
-                                       kReferenceAndUpdate);
+    TemporalLayers::FrameConfig tl_config(
+        kReferenceAndUpdate, kReferenceAndUpdate, kReferenceAndUpdate);
+    tl_config.pattern_idx = static_cast<int>(kTl1);
+    return tl_config;
   }
 
   const int64_t now_ms = clock_->TimeInMilliseconds();
   if (target_framerate_.value_or(0) > 0 &&
       encode_framerate_.Rate(now_ms).value_or(0) > *target_framerate_) {
     // Max framerate exceeded, drop frame.
     return TemporalLayers::FrameConfig(kNone, kNone, kNone);
   }
 
   if (stats_.first_frame_time_ms_ == -1)
     stats_.first_frame_time_ms_ = now_ms;
 
   int64_t unwrapped_timestamp = time_wrap_handler_.Unwrap(timestamp);
-  enum TemporalLayerState { kDrop, kTl0, kTl1, kTl1Sync };
-  enum TemporalLayerState layer_state = kDrop;
+  TemporalLayerState layer_state = kDrop;
   if (active_layer_ == -1 ||
       layers_[active_layer_].state != TemporalLayer::State::kDropped) {
     if (last_emitted_tl0_timestamp_ != -1 &&
         (unwrapped_timestamp - last_emitted_tl0_timestamp_) / 90 >
             kMaxFrameIntervalMs) {
       // Too long time has passed since the last frame was emitted, cancel
       // enough debt to allow a single frame.
       layers_[0].debt_bytes_ = max_debt_bytes_ - 1;
     }
     if (layers_[0].debt_bytes_ > max_debt_bytes_) {
@@ skipping 33 matching lines @@
   int64_t ts_diff;
   if (last_timestamp_ == -1) {
     ts_diff = kOneSecond90Khz / capture_framerate_.value_or(*target_framerate_);
   } else {
     ts_diff = unwrapped_timestamp - last_timestamp_;
   }
   // Make sure both frame droppers leak out bits.
   layers_[0].UpdateDebt(ts_diff / 90);
   layers_[1].UpdateDebt(ts_diff / 90);
   last_timestamp_ = timestamp;
+  TemporalLayers::FrameConfig tl_config;
   // TODO(pbos): Consider referencing but not updating the 'alt' buffer for all
   // layers.
   switch (layer_state) {
     case kDrop:
-      return TemporalLayers::FrameConfig(kNone, kNone, kNone);
+      tl_config = TemporalLayers::FrameConfig(kNone, kNone, kNone);
+      break;
     case kTl0:
       // TL0 only references and updates 'last'.
-      return TemporalLayers::FrameConfig(kReferenceAndUpdate, kNone, kNone);
+      tl_config =
+          TemporalLayers::FrameConfig(kReferenceAndUpdate, kNone, kNone);
+      break;
     case kTl1:
       // TL1 references both 'last' and 'golden' but only updates 'golden'.
-      return TemporalLayers::FrameConfig(kReference, kReferenceAndUpdate,
-                                         kNone);
+      tl_config =
+          TemporalLayers::FrameConfig(kReference, kReferenceAndUpdate, kNone);
+      break;
     case kTl1Sync:
       // Predict from only TL0 to allow participants to switch to the high
       // bitrate stream. Updates 'golden' so that TL1 can continue to refer to
       // and update 'golden' from this point on.
-      return TemporalLayers::FrameConfig(kReference, kUpdate, kNone,
-                                         kLayerSync);
+      tl_config = TemporalLayers::FrameConfig(kReference, kUpdate, kNone);
+      break;
   }
-  RTC_NOTREACHED();
-  return TemporalLayers::FrameConfig(kNone, kNone, kNone);
+
+  tl_config.pattern_idx = static_cast<int>(layer_state);
+  return tl_config;
 }
 
 std::vector<uint32_t> ScreenshareLayers::OnRatesUpdated(int bitrate_kbps,
                                                         int max_bitrate_kbps,
                                                         int framerate) {
   RTC_DCHECK_GT(framerate, 0);
   if (!target_framerate_) {
     // First OnRatesUpdated() is called during construction, with the configured
     // targets as parameters.
     target_framerate_.emplace(framerate);
@@ skipping 61 matching lines @@
     bool frame_is_keyframe,
     const TemporalLayers::FrameConfig& tl_config,
     CodecSpecificInfoVP8* vp8_info,
     uint32_t timestamp) {
   int64_t unwrapped_timestamp = time_wrap_handler_.Unwrap(timestamp);
   if (number_of_temporal_layers_ == 1) {
     vp8_info->temporalIdx = kNoTemporalIdx;
     vp8_info->layerSync = false;
     vp8_info->tl0PicIdx = kNoTl0PicIdx;
   } else {
-    // TODO(pbos): Add active_layer_ to TemporalLayers::FrameConfig (as
-    // pattern_idx) and make function const.
-    RTC_DCHECK_NE(-1, active_layer_);
-    vp8_info->temporalIdx = active_layer_;
+    TemporalLayerState layer_state =
+        static_cast<TemporalLayerState>(tl_config.pattern_idx);
+    switch (layer_state) {
+      case kDrop:
+        RTC_NOTREACHED();
+        break;
+      case kTl0:
+        vp8_info->temporalIdx = 0;
+        break;
+      case kTl1:
+      case kTl1Sync:
+        vp8_info->temporalIdx = 1;
+        break;
+    }
     if (frame_is_keyframe) {
       vp8_info->temporalIdx = 0;
       last_sync_timestamp_ = unwrapped_timestamp;
     } else if (last_base_layer_sync_ && vp8_info->temporalIdx != 0) {
       // Regardless of pattern the frame after a base layer sync will always
       // be a layer sync.
       last_sync_timestamp_ = unwrapped_timestamp;
     }
     vp8_info->layerSync = last_sync_timestamp_ != -1 &&
                           last_sync_timestamp_ == unwrapped_timestamp;
@@ skipping 147 matching lines @@
       RTC_HISTOGRAM_COUNTS_10000("WebRTC.Video.Screenshare.Layer1.Qp",
                                  stats_.tl1_qp_sum_ / stats_.num_tl1_frames_);
       RTC_HISTOGRAM_COUNTS_10000(
           "WebRTC.Video.Screenshare.Layer1.TargetBitrate",
           stats_.tl1_target_bitrate_sum_ / stats_.num_tl1_frames_);
     }
   }
 }
 
 }  // namespace webrtc