RTCPeerConnection.addTrack and removeTrack added (behind flag)

chrome/browser/media/webrtc/webrtc_rtp_browsertest.cc

 // Copyright 2017 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+#include <string>
+#include <vector>
+
 #include "base/command_line.h"
 #include "chrome/browser/media/webrtc/webrtc_browsertest_base.h"
 #include "content/public/common/content_switches.h"
 #include "media/base/media_switches.h"
 
 static const char kMainWebrtcTestHtmlPage[] = "/webrtc/webrtc_jsep01_test.html";
 
 class WebRtcRtpBrowserTest : public WebRtcTestBase {
  public:
   WebRtcRtpBrowserTest() : left_tab_(nullptr), right_tab_(nullptr) {}
@@ skipping 41 matching lines @@
   CreateAndAddStreams(left_tab_, 3);
 
   SetupPeerconnectionWithoutLocalStream(right_tab_);
   CreateAndAddStreams(right_tab_, 1);
 
   NegotiateCall(left_tab_, right_tab_);
 
   VerifyRtpReceivers(left_tab_, 2);
   VerifyRtpReceivers(right_tab_, 6);
 }
+
+IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest, AddAndRemoveTracksWithoutStream) {
+  StartServerAndOpenTabs();
+
+  SetupPeerconnectionWithoutLocalStream(left_tab_);
+  SetupPeerconnectionWithoutLocalStream(right_tab_);
+
+  std::vector<std::string> ids =
+      CreateAndAddAudioAndVideoTrack(left_tab_, StreamArgumentType::NO_STREAM);
+  std::string audio_stream_id = ids[0];
+  std::string audio_track_id = ids[1];
+  std::string video_stream_id = ids[2];
+  std::string video_track_id = ids[3];
+  EXPECT_NE("null", audio_stream_id);

[Taylor_Brandstetter, 2017/07/06 22:45:14]

I think we should fix this before we ship the feature (meaning, if you add a
track without a stream, no stream should get generated).

[hbos_chromium, 2017/07/07 12:07:47]

SGTM. Will fix before unflagging. Added a TODO.

The EXPECT_NE changed to EXPECT_EQ because now that I don't parse the SDP I
don't know the resulting stream. Test updated to test both sender/receiver and
local/remote streams.

+  EXPECT_NE("null", audio_track_id);
+  EXPECT_NE("null", video_stream_id);
+  EXPECT_NE("null", video_track_id);
+  EXPECT_NE(audio_stream_id, video_stream_id);
+
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 2);
+  VerifyRtpReceivers(right_tab_, 2);
+
+  RemoveTrack(left_tab_, audio_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 1);
+  VerifyRtpReceivers(right_tab_, 1);
+
+  RemoveTrack(left_tab_, video_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 0);
+  VerifyRtpReceivers(right_tab_, 0);
+}
+
+IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest,
+                       AddAndRemoveTracksWithSharedStream) {
+  StartServerAndOpenTabs();
+
+  SetupPeerconnectionWithoutLocalStream(left_tab_);
+  SetupPeerconnectionWithoutLocalStream(right_tab_);
+
+  std::vector<std::string> ids = CreateAndAddAudioAndVideoTrack(
+      left_tab_, StreamArgumentType::SHARED_STREAM);
+  std::string audio_stream_id = ids[0];
+  std::string audio_track_id = ids[1];
+  std::string video_stream_id = ids[2];
+  std::string video_track_id = ids[3];
+  EXPECT_NE("null", audio_stream_id);
+  EXPECT_EQ(audio_stream_id, video_stream_id);
+  EXPECT_NE("null", audio_track_id);
+  EXPECT_NE("null", video_track_id);
+
+  NegotiateCall(left_tab_, right_tab_);
+  // TODO(hbos): Here and in other "AddAndRemoveTracks" tests: when ontrack and
+  // ended events are supported, verify that these are fired on the remote side
+  // when tracks are added and removed. https://crbug.com/webrtc/7933
+  VerifyRtpSenders(left_tab_, 2);
+  VerifyRtpReceivers(right_tab_, 2);
+
+  RemoveTrack(left_tab_, audio_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 1);
+  VerifyRtpReceivers(right_tab_, 1);
+
+  RemoveTrack(left_tab_, video_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 0);
+  VerifyRtpReceivers(right_tab_, 0);
+}
+
+IN_PROC_BROWSER_TEST_F(WebRtcRtpBrowserTest,
+                       AddAndRemoveTracksWithIndividualStreams) {
+  StartServerAndOpenTabs();
+
+  SetupPeerconnectionWithoutLocalStream(left_tab_);
+  SetupPeerconnectionWithoutLocalStream(right_tab_);
+
+  std::vector<std::string> ids = CreateAndAddAudioAndVideoTrack(
+      left_tab_, StreamArgumentType::INDIVIDUAL_STREAMS);
+  std::string audio_stream_id = ids[0];
+  std::string audio_track_id = ids[1];
+  std::string video_stream_id = ids[2];
+  std::string video_track_id = ids[3];
+  EXPECT_NE("null", audio_stream_id);
+  EXPECT_NE("null", audio_track_id);
+  EXPECT_NE("null", video_stream_id);
+  EXPECT_NE("null", video_track_id);
+  EXPECT_NE(audio_stream_id, video_stream_id);
+
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 2);
+  VerifyRtpReceivers(right_tab_, 2);
+
+  RemoveTrack(left_tab_, audio_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 1);
+  VerifyRtpReceivers(right_tab_, 1);
+
+  RemoveTrack(left_tab_, video_track_id);
+  NegotiateCall(left_tab_, right_tab_);
+  VerifyRtpSenders(left_tab_, 0);
+  VerifyRtpReceivers(right_tab_, 0);
+}