Bug 168647 – [GTK] Several WebRTC tests failing since r212812

• Component: WebKitGTK
• Status: RESOLVED DUPLICATE

[Carlos Garcia Campos]

fast/mediastream/RTCPeerConnection-more-media-to-negotiate.html
imported/w3c/web-platform-tests/webrtc/datachannel-emptystring.html
imported/w3c/web-platform-tests/webrtc/no-media-call.html
imported/w3c/web-platform-tests/webrtc/promises-call.html

[Carlos Garcia Campos]

r212699 has just been rolled out in r212704. Closing then, sorry for the noise.

[Carlos Garcia Campos]

Patch landed again with the same failures.

[Carlos Alberto Lopez Perez]

Please, when reporting a bug about new failures, CC the authors of the commit that introduced that to let them know about the issue.

[youenn fablet]

Sorry about breaking these tests here.
Mac and GTK ports no longer use the same WebRTC mock backends, hence the test differences. I tried to move GTK test expectations to the GTK platform folder but failed apparently to do so for some tests.
I guess the simplest approach might be to do the same for these 4 tests.

[youenn fablet]

I will land https://bugs.webkit.org/show_bug.cgi?id=167293 which should fix some of the failing tests.

[Alejandro G. Castro]

That patch fixed the tests https://trac.webkit.org/changeset/212817

Thanks

[Alejandro G. Castro]

My mistake, some of this tests are inside the general TestExpectations and some in the platform, they are not really passing. We have to remove some from the GTK expectations and just check those in this bug. The other ones were added here to the TestExpectations:

https://trac.webkit.org/changeset/212848

[Carlos Alberto Lopez Perez]

The 3 tests from imported/w3c/web-platform-tests/webrtc were failing also before r212812, but the expectation for them was just wrong.

And fast/mediastream/RTCPeerConnection-more-media-to-negotiate.html is passing with the general expectation but not with the one for GTK+ that got copied from the general at r212338 <https://trac.webkit.org/r212338>


$ diff -u LayoutTests/fast/mediastream/RTCPeerConnection-more-media-to-negotiate-expected.txt LayoutTests/platform/gtk/fast/mediastream/RTCPeerConnection-more-media-to-negotiate-expected.txt
--- LayoutTests/fast/mediastream/RTCPeerConnection-more-media-to-negotiate-expected.txt	2017-02-16 03:33:41.255592879 +0100
+++ LayoutTests/platform/gtk/fast/mediastream/RTCPeerConnection-more-media-to-negotiate-expected.txt	2017-02-23 16:32:14.768622317 +0100
@@ -18,6 +18,7 @@
 PASS pc.getTransceivers().length is 2
 Create an answer for the audio only offer
 Set answer as local description
+PASS negotiationneededevent fired: There is local media not negotiated that needs to be negotiated in a follow-up offer
 PASS Answer set
 PASS successfullyParsed is true


I'm not 100% sure... but I'm going to assume that the general is the right one, remove the specific GTK+ expectation and mark this test as passing.

[Carlos Alberto Lopez Perez]

I have landed https://trac.webkit.org/changeset/212884 unskipping many mediastream/webrtc tests for GTK that were skipped on the general and also marking the expected failures as expected.


fast/mediastream/RTCPeerConnection-more-media-to-negotiate.html has been rebaselined to the general expectation

And the bug tracking the failures on imported/w3c/web-platform-tests/webrtc/datachannel-emptystring.html imported/w3c/web-platform-tests/webrtc/no-media-call.html and imported/w3c/web-platform-tests/webrtc/promises-call.html is 168736 

We can close this

*** This bug has been marked as a duplicate of bug 168736 ***