From efd01fd354f66efa5d7429ad4f3ab3b74ab4a5b4 Mon Sep 17 00:00:00 2001
From: "marpan@webrtc.org"
 <marpan@webrtc.org@4adac7df-926f-26a2-2b94-8c16560cd09d>
Date: Wed, 18 Apr 2012 15:56:34 +0000
Subject: [PATCH] Removing unused code from QMVideoSettingsCallback. This code
 is not needed as of the change in r1950. Review URL:
 https://webrtc-codereview.appspot.com/492010

git-svn-id: http://webrtc.googlecode.com/svn/trunk@2052 4adac7df-926f-26a2-2b94-8c16560cd09d
---
 src/video_engine/vie_encoder.cc | 44 +++------------------------------
 1 file changed, 4 insertions(+), 40 deletions(-)

diff --git a/src/video_engine/vie_encoder.cc b/src/video_engine/vie_encoder.cc
index c5f3813340..1b2b65a629 100644
--- a/src/video_engine/vie_encoder.cc
+++ b/src/video_engine/vie_encoder.cc
@@ -28,12 +28,7 @@ namespace webrtc {
 
 class QMVideoSettingsCallback : public VCMQMSettingsCallback {
  public:
-  QMVideoSettingsCallback(WebRtc_Word32 engine_id,
-                          WebRtc_Word32 channel_id,
-                          VideoProcessingModule* vpm,
-                          VideoCodingModule* vcm,
-                          WebRtc_Word32 num_of_cores,
-                          WebRtc_Word32 max_payload_length);
+  explicit QMVideoSettingsCallback(VideoProcessingModule* vpm);
   ~QMVideoSettingsCallback();
 
   // Update VPM with QM (quality modes: frame size & frame rate) settings.
@@ -41,15 +36,8 @@ class QMVideoSettingsCallback : public VCMQMSettingsCallback {
                                    const WebRtc_UWord32 width,
                                    const WebRtc_UWord32 height);
 
-  void SetMaxPayloadLength(WebRtc_Word32 max_payload_length);
-
  private:
-  WebRtc_Word32 engine_id_;
-  WebRtc_Word32 channel_id_;
   VideoProcessingModule* vpm_;
-  VideoCodingModule* vcm_;
-  WebRtc_Word32 num_cores_;
-  WebRtc_Word32 max_payload_length_;
 };
 
 
@@ -136,13 +124,7 @@ bool ViEEncoder::Init() {
   if (qm_callback_) {
     delete qm_callback_;
   }
-  qm_callback_ = new QMVideoSettingsCallback(
-      engine_id_,
-      channel_id_,
-      &vpm_,
-      &vcm_,
-      number_of_cores_,
-      default_rtp_rtcp_.MaxDataPayloadLength());
+  qm_callback_ = new QMVideoSettingsCallback(&vpm_);
 
 #ifdef VIDEOCODEC_VP8
   VideoCodec video_codec;
@@ -360,8 +342,6 @@ WebRtc_Word32 ViEEncoder::SetEncoder(const webrtc::VideoCodec& video_codec) {
   WebRtc_UWord16 max_data_payload_length =
       default_rtp_rtcp_.MaxDataPayloadLength();
 
-  qm_callback_->SetMaxPayloadLength(max_data_payload_length);
-
   if (vcm_.RegisterSendCodec(&video_codec, number_of_cores_,
                              max_data_payload_length) != VCM_OK) {
     WEBRTC_TRACE(webrtc::kTraceError, webrtc::kTraceVideo,
@@ -874,19 +854,8 @@ ViEFileRecorder& ViEEncoder::GetOutgoingFileRecorder() {
   return file_recorder_;
 }
 
-QMVideoSettingsCallback::QMVideoSettingsCallback(
-    WebRtc_Word32 engine_id,
-    WebRtc_Word32 channel_id,
-    VideoProcessingModule* vpm,
-    VideoCodingModule* vcm,
-    WebRtc_Word32 num_cores,
-    WebRtc_Word32 max_payload_length)
-    : engine_id_(engine_id),
-      channel_id_(channel_id),
-      vpm_(vpm),
-      vcm_(vcm),
-      num_cores_(num_cores),
-      max_payload_length_(max_payload_length) {
+QMVideoSettingsCallback::QMVideoSettingsCallback(VideoProcessingModule* vpm)
+    : vpm_(vpm) {
 }
 
 QMVideoSettingsCallback::~QMVideoSettingsCallback() {
@@ -899,9 +868,4 @@ WebRtc_Word32 QMVideoSettingsCallback::SetVideoQMSettings(
   return vpm_->SetTargetResolution(width, height, frame_rate);
 }
 
-void QMVideoSettingsCallback::SetMaxPayloadLength(
-    WebRtc_Word32 max_payload_length) {
-  max_payload_length_ = max_payload_length;
-}
-
 }  // namespace webrtc