commit c7199c2d0b4a70dedc5bb9df5d8d5c15ec17c156
author ivica <ivica@webrtc.org> Wed Oct 07 06:43:33 2015 -0700
committer Commit bot <commit-bot@chromium.org> Wed Oct 07 13:43:43 2015 +0000
tree 25b98e617811d695c9e3401c3dfa433c3218c143
parent 78543284d0010efa4270ff9c102b673593c4827b

Read the number of TLs for VP9 too + cleanup

In video_sender.cc, properly read the number of temporal layers for VP9 too.

Also, some cleanup in video_loopback.cc and video_quality_test.h.

Review URL: https://codereview.webrtc.org/1351693005

Cr-Commit-Position: refs/heads/master@{#10201}

webrtc/modules/video_coding/main/source/video_sender.cc

diff --git a/webrtc/modules/video_coding/main/source/video_sender.cc b/webrtc/modules/video_coding/main/source/video_sender.cc
index c5317ec..a5ef22a 100644
--- a/webrtc/modules/video_coding/main/source/video_sender.cc
+++ b/webrtc/modules/video_coding/main/source/video_sender.cc
@@ -106,9 +106,15 @@
     return VCM_CODEC_ERROR;
   }
 
-  int numLayers = (sendCodec->codecType != kVideoCodecVP8)
-                      ? 1
-                      : sendCodec->codecSpecific.VP8.numberOfTemporalLayers;
+  int numLayers;
+  if (sendCodec->codecType == kVideoCodecVP8) {
+    numLayers = sendCodec->codecSpecific.VP8.numberOfTemporalLayers;
+  } else if (sendCodec->codecType == kVideoCodecVP9) {
+    numLayers = sendCodec->codecSpecific.VP9.numberOfTemporalLayers;
+  } else {
+    numLayers = 1;
+  }
+
   // If we have screensharing and we have layers, we disable frame dropper.
   bool disable_frame_dropper =
       numLayers > 1 && sendCodec->mode == kScreensharing;

webrtc/video/video_loopback.cc

diff --git a/webrtc/video/video_loopback.cc b/webrtc/video/video_loopback.cc
index 0dceea8..0c06f85 100644
--- a/webrtc/video/video_loopback.cc
+++ b/webrtc/video/video_loopback.cc
@@ -10,18 +10,14 @@
 
 #include <stdio.h>
 
-#include <map>
-
 #include "gflags/gflags.h"
 #include "testing/gtest/include/gtest/gtest.h"
 
 #include "webrtc/test/field_trial.h"
 #include "webrtc/test/run_test.h"
-#include "webrtc/typedefs.h"
 #include "webrtc/video/video_quality_test.h"
 
 namespace webrtc {
-
 namespace flags {
 
 DEFINE_int32(width, 640, "Video width.");

webrtc/video/video_quality_test.h

diff --git a/webrtc/video/video_quality_test.h b/webrtc/video/video_quality_test.h
index f47140e..7b62fb3 100644
--- a/webrtc/video/video_quality_test.h
+++ b/webrtc/video/video_quality_test.h
@@ -34,8 +34,8 @@
       int max_bitrate_bps;
       std::string codec;
       size_t num_temporal_layers;
-
       int min_transmit_bps;
+
       Call::Config::BitrateConfig call_bitrate_config;
       size_t tl_discard_threshold;
       bool send_side_bwe;