commit 0f5400acfa40b4ecbf71bf44d925ea6ec9d3401d
author Steve Anton <steveanton@webrtc.org> Tue Jul 17 14:25:36 2018 -0700
committer Steve Anton <steveanton@webrtc.org> Tue Jul 17 23:56:04 2018 +0000
tree 5bacba4ef4393aa7b6fa5bfc5bbeb650b62b2a9f
parent dbdb3a00797d28c2e21191efd859d01b2533c003

[Unified Plan] Implement FiredDirection for RtpTransceiver

Bug: webrtc:9236
Change-Id: Ib5a8215f3762f35b68d2a285c7d676f93f1212c5
Reviewed-on: https://webrtc-review.googlesource.com/88921
Reviewed-by: Henrik Boström <hbos@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24010}

pc/peerconnectioninterface_unittest.cc

diff --git a/pc/peerconnectioninterface_unittest.cc b/pc/peerconnectioninterface_unittest.cc
index 25e7a2c..c0f2c73 100644
--- a/pc/peerconnectioninterface_unittest.cc
+++ b/pc/peerconnectioninterface_unittest.cc
@@ -3292,15 +3292,7 @@
 
   // Create and set the updated remote SDP.
   CreateAndSetRemoteOffer(kSdpStringWithStream1PlanB);
-  if (sdp_semantics_ == SdpSemantics::kPlanB) {
-    EXPECT_EQ(observer_.num_added_tracks_, 2);
-  } else {
-    // With Unified Plan, OnAddTrack will fire every time SetRemoteDescription
-    // is called until the offer/answer exchange is complete. So in this case
-    // OnAddTrack is fired twice for the first audio track plus the one time
-    // for the video track.
-    EXPECT_EQ(observer_.num_added_tracks_, 3);
-  }
+  EXPECT_EQ(observer_.num_added_tracks_, 2);
   EXPECT_EQ(observer_.last_added_track_label_, kVideoTracks[0]);
 }