Bug 203421 – Add a test verifying video plays well when switching from MediaStreams to regular videos

• Status: RESOLVED
• Resolution: FIXED
• Component: WebRTC

[youenn fablet]

Add a test verifying video plays well when switching from MediaStreams to regular videos

[youenn fablet]

Created attachment 381934 [details]
Patch

[Geoffrey Garen]

Comment on attachment 381934 [details]
Patch

r=me

[Eric Carlson]

Comment on attachment 381934 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=381934&action=review

> LayoutTests/fast/mediastream/stream-switch.html:17
> +        localVideo.src = "../../media/content/audio-tracks.mp4";

You can make this work cross platform by including "../../media/media-file.js" and using "localVideo.src = findMediaFile('video', 'content/test')"

[youenn fablet]

Created attachment 382722 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 382722 [details]
Patch for landing

Clearing flags on attachment: 382722

Committed r251982: <https://trac.webkit.org/changeset/251982>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/56860567>

[Philippe Normand]

(In reply to Eric Carlson from comment #3)
> Comment on attachment 381934 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=381934&action=review
> 
> > LayoutTests/fast/mediastream/stream-switch.html:17
> > +        localVideo.src = "../../media/content/audio-tracks.mp4";
> 
> You can make this work cross platform by including
> "../../media/media-file.js" and using "localVideo.src =
> findMediaFile('video', 'content/test')"

Turns out this didn't really work, see https://github.com/WebKit/WebKit/pull/3737 :)