Remove dead code in vp8_impl files.

webrtc/common_types.h

 /*
  *  Copyright (c) 2012 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.
  */
 
@@ skipping 506 matching lines @@
                      // packet losses, but packets within a frame subsequent
                      // to a loss can't be decoded.
   kResilientFrames   // Same as kResilientStream but with added resilience
                      // within a frame.
 };
 
 class TemporalLayersFactory;
 // VP8 specific
 struct VideoCodecVP8 {
   bool pictureLossIndicationOn;
-  bool feedbackModeOn;

[brandtr, 2017/03/17 08:26:13]

Need to update in Chromium too:
https://cs.chromium.org/chromium/src/content/renderer/media/gpu/rtc_video_dec...
https://cs.chromium.org/chromium/src/content/renderer/media/gpu/rtc_video_dec...

[nisse-webrtc, 2017/03/17 08:54:48]

My plan was to first delete all use of this variable, then drop it in chrome,
and finally delete it here. pictureLossIndicationOn is also unused, and could be
deleted in chromium right away, and then deleted here.

[pbos-webrtc, 2017/03/17 17:48:55]

Thanks, removing in https://codereview.chromium.org/2757913002/.

[pbos-webrtc, 2017/03/17 17:48:55]

Ack, can do that separately though.

   VideoCodecComplexity complexity;
   VP8ResilienceMode resilience;
   unsigned char numberOfTemporalLayers;
   bool denoisingOn;
   bool errorConcealmentOn;
   bool automaticResizeOn;
   bool frameDroppingOn;
   int keyFrameInterval;
   TemporalLayersFactory* tl_factory;
 };
@@ skipping 360 matching lines @@
 enum class RtcpMode { kOff, kCompound, kReducedSize };
 
 enum NetworkState {
   kNetworkUp,
   kNetworkDown,
 };
 
 }  // namespace webrtc
 
 #endif  // WEBRTC_COMMON_TYPES_H_