Bug 1371362: Remove unused audio and video stream counters of PeerConnectionImpl r=bwc

MozReview-Commit-ID: AtZRVqCyF4A

--HG--
extra : rebase_source : 5c395ecd15a8403de8181675a61f2545a5a14299
This commit is contained in:
Paul Ellenbogen 2017-06-08 11:17:08 -07:00
parent a667d4ec7d
commit ad7b0fb2a3
2 changed files with 0 additions and 10 deletions

View File

@ -309,8 +309,6 @@ PeerConnectionImpl::PeerConnectionImpl(const GlobalObject* aGlobal)
, mForceIceTcp(false)
, mMedia(nullptr)
, mUuidGen(MakeUnique<PCUuidGenerator>())
, mNumAudioStreams(0)
, mNumVideoStreams(0)
, mHaveConfiguredCodecs(false)
, mHaveDataStream(false)
, mAddCandidateErrorCount(0)
@ -2340,7 +2338,6 @@ PeerConnectionImpl::AddTrack(MediaStreamTrack& aTrack,
if (NS_FAILED(res)) {
return res;
}
mNumAudioStreams++;
}
if (aTrack.AsVideoStreamTrack()) {
@ -2354,7 +2351,6 @@ PeerConnectionImpl::AddTrack(MediaStreamTrack& aTrack,
if (NS_FAILED(res)) {
return res;
}
mNumVideoStreams++;
}
OnNegotiationNeeded();
return NS_OK;

View File

@ -788,12 +788,6 @@ private:
// Start time of call used for Telemetry
mozilla::TimeStamp mStartTime;
// Temporary: used to prevent multiple audio streams or multiple video streams
// in a single PC. This is tied up in the IETF discussion around proper
// representation of multiple streams in SDP, and strongly related to
// Bug 840728.
int mNumAudioStreams;
int mNumVideoStreams;
bool mHaveConfiguredCodecs;
bool mHaveDataStream;