diff --git a/api/jsep_session_description.h b/api/jsep_session_description.h index 0b65734ea9..c51abd57d4 100644 --- a/api/jsep_session_description.h +++ b/api/jsep_session_description.h @@ -71,9 +71,6 @@ class JsepSessionDescription : public SessionDescriptionInterface { size_t mediasection_index) const; virtual bool ToString(std::string* out) const; - static const int kDefaultVideoCodecId; - static const char kDefaultVideoCodecName[]; - private: std::unique_ptr description_; std::string session_id_; diff --git a/pc/jsep_session_description.cc b/pc/jsep_session_description.cc index 4c57396f08..885c1eb310 100644 --- a/pc/jsep_session_description.cc +++ b/pc/jsep_session_description.cc @@ -121,9 +121,6 @@ void UpdateConnectionAddress( } // namespace -const int JsepSessionDescription::kDefaultVideoCodecId = 100; -const char JsepSessionDescription::kDefaultVideoCodecName[] = "VP8"; - // TODO(steveanton): Remove this default implementation once Chromium has been // updated. SdpType SessionDescriptionInterface::GetType() const { diff --git a/pc/webrtc_sdp_unittest.cc b/pc/webrtc_sdp_unittest.cc index 37464425c6..0e1e62e53f 100644 --- a/pc/webrtc_sdp_unittest.cc +++ b/pc/webrtc_sdp_unittest.cc @@ -1317,8 +1317,7 @@ class WebRtcSdpTest : public ::testing::Test { 1, "AES_CM_128_HMAC_SHA1_80", "inline:d0RmdmcmVCspeEc3QGZiNWpVLFJhQX1cfHAwJSoj|2^20|1:32", "")); video->set_protocol(cricket::kMediaProtocolSavpf); - video->AddCodec( - VideoCodec(120, JsepSessionDescription::kDefaultVideoCodecName)); + video->AddCodec(VideoCodec(120, "VP8")); return video; } @@ -1969,8 +1968,7 @@ class WebRtcSdpTest : public ::testing::Test { ASSERT_TRUE(vcd); ASSERT_FALSE(vcd->codecs().empty()); cricket::VideoCodec vp8 = vcd->codecs()[0]; - EXPECT_STREQ(webrtc::JsepSessionDescription::kDefaultVideoCodecName, - vp8.name.c_str()); + EXPECT_EQ(vp8.name, "VP8"); EXPECT_EQ(101, vp8.id); EXPECT_TRUE(vp8.HasFeedbackParam(cricket::FeedbackParam( cricket::kRtcpFbParamLntf, cricket::kParamValueEmpty)));