Title: [214890] trunk/LayoutTests
Revision
214890
Author
[email protected]
Date
2017-04-04 12:24:31 -0700 (Tue, 04 Apr 2017)

Log Message

webrtc/peer-connection-audio-mute.html is flaky
https://bugs.webkit.org/show_bug.cgi?id=170451

Patch by Youenn Fablet <[email protected]> on 2017-04-04
Reviewed by Eric Carlson.

* TestExpectations: Removed flaky expectation for webrtc/peer-connection-audio-mute.html.
* webrtc/peer-connection-remote-audio-mute.html: Added timer to ensure setting enable propagates.
Decreasing time spent at detecting hum or not.
* webrtc/peer-connection-remote-audio-mute2.html: Ditto (but keeping flakiness expectation for now).

Modified Paths

Diff

Modified: trunk/LayoutTests/ChangeLog (214889 => 214890)


--- trunk/LayoutTests/ChangeLog	2017-04-04 19:14:02 UTC (rev 214889)
+++ trunk/LayoutTests/ChangeLog	2017-04-04 19:24:31 UTC (rev 214890)
@@ -1,3 +1,15 @@
+2017-04-04  Youenn Fablet  <[email protected]>
+
+        webrtc/peer-connection-audio-mute.html is flaky
+        https://bugs.webkit.org/show_bug.cgi?id=170451
+
+        Reviewed by Eric Carlson.
+
+        * TestExpectations: Removed flaky expectation for webrtc/peer-connection-audio-mute.html.
+        * webrtc/peer-connection-remote-audio-mute.html: Added timer to ensure setting enable propagates.
+        Decreasing time spent at detecting hum or not.
+        * webrtc/peer-connection-remote-audio-mute2.html: Ditto (but keeping flakiness expectation for now).
+
 2017-04-04  Ryan Haddad  <[email protected]>
 
         Mark fast/images/animated-gif-webkit-transform.html as flaky.

Modified: trunk/LayoutTests/TestExpectations (214889 => 214890)


--- trunk/LayoutTests/TestExpectations	2017-04-04 19:14:02 UTC (rev 214889)
+++ trunk/LayoutTests/TestExpectations	2017-04-04 19:24:31 UTC (rev 214890)
@@ -716,7 +716,6 @@
 webrtc/video-getParameters.html [ Failure ]
 webrtc/peer-connection-audio-mute2.html [ Pass Failure ]
 webrtc/peer-connection-remote-audio-mute2.html [ Pass Failure ]
-webrtc/peer-connection-remote-audio-mute.html [ Pass Failure ]
 webkit.org/b/170178 webrtc/video-replace-track-to-null.html [ Pass Failure ]
 fast/mediastream/getUserMedia-webaudio.html [ Skip ]
 fast/mediastream/RTCPeerConnection-closed-state.html [ Skip ]

Modified: trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute.html (214889 => 214890)


--- trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute.html	2017-04-04 19:14:02 UTC (rev 214889)
+++ trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute.html	2017-04-04 19:24:31 UTC (rev 214890)
@@ -30,13 +30,14 @@
                     };
                 });
             }).then(() => {
-                return analyseAudio(remoteStream, 500).then((results) => {
+                return analyseAudio(remoteStream, 200).then((results) => {
                     assert_true(results.heardHum, "heard hum from remote enabled track");
                 });
             }).then(() => {
                 remoteTrack.enabled = false;
+                return waitFor(100);
             }).then(() => {
-                return analyseAudio(remoteStream, 500).then((results) => {
+                return analyseAudio(remoteStream, 200).then((results) => {
                     assert_false(results.heardHum, "not heard hum from remote disabled track");
                 });
             });

Modified: trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute2.html (214889 => 214890)


--- trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute2.html	2017-04-04 19:14:02 UTC (rev 214889)
+++ trunk/LayoutTests/webrtc/peer-connection-remote-audio-mute2.html	2017-04-04 19:24:31 UTC (rev 214890)
@@ -35,6 +35,7 @@
                 });
             }).then(() => {
                 remoteTrack.enabled = false;
+                return waitFor(100);
             }).then(() => {
                 return analyseAudio(remoteStream, 500).then((results) => {
                     assert_false(results.heardHum, "not heard hum from remote disabled track");
@@ -41,6 +42,7 @@
                 });
             }).then(() => {
                 remoteTrack.enabled = true;
+                return waitFor(100);
             }).then(() => {
                 return analyseAudio(remoteStream, 500).then((results) => {
                     assert_true(results.heardHum, "heard hum from remote reenabled track");
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to