Bug 1414628 - ConfigureCodec: Remove private field 'mHardwareH264Enabled' r?jesup draft
authorSylvestre Ledru <sledru@mozilla.com>
Sun, 05 Nov 2017 16:56:19 +0100
changeset 693325 2d9cd1ed3b165dabc1f10c51eac67d5998c01b82
parent 693319 849017ffe2976f7721c20c3625d8fafeb3cce8dd
child 693326 600fbe2035142160b4ee93d6b26e25d0f5f0c4f6
child 693327 da7b66f36207cf90611dddc7bf52bddacdf0c3fc
push id87756
push userbmo:sledru@mozilla.com
push dateSun, 05 Nov 2017 15:56:49 +0000
reviewersjesup
bugs1414628
milestone58.0a1
Bug 1414628 - ConfigureCodec: Remove private field 'mHardwareH264Enabled' r?jesup MozReview-Commit-ID: 83U02hqgW6d
media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp
--- a/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp
+++ b/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp
@@ -824,17 +824,16 @@ class CompareCodecPriority {
 
   private:
     std::string mPreferredCodec;
 };
 
 class ConfigureCodec {
   public:
     explicit ConfigureCodec(nsCOMPtr<nsIPrefBranch>& branch) :
-      mHardwareH264Enabled(false),
       mHardwareH264Supported(false),
       mSoftwareH264Enabled(false),
       mH264Enabled(false),
       mVP9Enabled(true),
       mVP9Preferred(false),
       mH264Level(13), // minimum suggested for WebRTC spec
       mH264MaxBr(0), // Unlimited
       mH264MaxMbps(0), // Unlimited
@@ -958,17 +957,16 @@ class ConfigureCodec {
         case SdpMediaSection::kText:
         case SdpMediaSection::kApplication:
         case SdpMediaSection::kMessage:
           {} // Nothing to configure for these.
       }
     }
 
   private:
-    bool mHardwareH264Enabled;
     bool mHardwareH264Supported;
     bool mSoftwareH264Enabled;
     bool mH264Enabled;
     bool mVP9Enabled;
     bool mVP9Preferred;
     int32_t mH264Level;
     int32_t mH264MaxBr;
     int32_t mH264MaxMbps;