Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(633)

Unified Diff: content/test/data/media/peerconnection-call.html

Issue 417203002: Add browser test to connect chormium sink to remote tracks, verify that no crash. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « content/browser/media/webrtc_browsertest.cc ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/test/data/media/peerconnection-call.html
diff --git a/content/test/data/media/peerconnection-call.html b/content/test/data/media/peerconnection-call.html
index cb12d34791ac5065e41f63107bb05d168ca7efe7..97aaf22d38cd698a2c303fff865725bf025b501e 100644
--- a/content/test/data/media/peerconnection-call.html
+++ b/content/test/data/media/peerconnection-call.html
@@ -136,6 +136,30 @@
detectVideoPlaying('remote-view-2', onRemoteStream2);
}
+ // First makes a call between pc1 and pc2, and then construct a new media
+ // stream using the remote audio and video tracks, connect the new media
+ // stream to a video element. These operations should not crash Chrome.
+ function ConnectChromiumSinkToRemoteAudioTrack() {
+ createConnections(null);
+ navigator.webkitGetUserMedia({audio: true, video: true},
+ addStreamToBothConnectionsAndNegotiate,
+ printGetUserMediaError);
+
+ detectVideoPlaying('remote-view-2', function() {
+ // Construct a new media stream with remote tracks.
+ var newStream = new webkitMediaStream();
+ newStream.addTrack(
+ gSecondConnection.getRemoteStreams()[0].getAudioTracks()[0]);
+ newStream.addTrack(
+ gSecondConnection.getRemoteStreams()[0].getVideoTracks()[0]);
+ var videoElement = document.createElement('video');
+
+ // No crash for this operation.
+ videoElement.src = URL.createObjectURL(newStream);
+ waitForVideo('remote-view-2');
+ });
+ }
+
// Test that we can setup call with an audio and video track and
// simulate that the remote peer don't support MSID.
function callWithoutMsidAndBundle() {
« no previous file with comments | « content/browser/media/webrtc_browsertest.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698