[iOS Release and Mac Release] imported/w3c/web-platform-tests/webrtc/RTCDTMFSender-ontonechange.https.html is flaky failing

[Jason Lawrence]

imported/w3c/web-platform-tests/webrtc/RTCDTMFSender-ontonechange.https.html

Description: 
This test is flaky failing on iOS release and Mac release

History:
https://results.webkit.org/?suite=layout-tests&test=imported%2Fw3c%2Fweb-platform-tests%2Fwebrtc%2FRTCDTMFSender-ontonechange.https.html

Diff:
--- /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/webrtc/RTCDTMFSender-ontonechange.https-expected.txt
+++ /Volumes/Data/slave/ios-simulator-13-release-tests-wk2/build/layout-test-results/imported/w3c/web-platform-tests/webrtc/RTCDTMFSender-ontonechange.https-actual.txt
@@ -1,14 +1,14 @@
 
-PASS insertDTMF() with default duration and intertoneGap should fire tonechange events at the expected time 
-PASS insertDTMF() with explicit duration and intertoneGap should fire tonechange events at the expected time 
-PASS insertDTMF('') should not fire any tonechange event, including for '' tone 
-PASS insertDTMF() with duration less than 40 should be clamped to 40 
-PASS insertDTMF() with interToneGap less than 30 should be clamped to 30 
-PASS insertDTMF with comma should delay next tonechange event for a constant 2000ms 
-PASS insertDTMF() with transceiver stopped in the middle should stop future tonechange events from firing 
-PASS Calling insertDTMF() in the middle of tonechange events should cause future tonechanges to be updated to new tones 
-PASS Calling insertDTMF() multiple times in the middle of tonechange events should cause future tonechanges to be updated the last provided tones 
-PASS Calling insertDTMF('') in the middle of tonechange events should stop future tonechange events from firing 
+FAIL insertDTMF() with default duration and intertoneGap should fire tonechange events at the expected time assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF() with explicit duration and intertoneGap should fire tonechange events at the expected time assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF('') should not fire any tonechange event, including for '' tone assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF() with duration less than 40 should be clamped to 40 assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF() with interToneGap less than 30 should be clamped to 30 assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF with comma should delay next tonechange event for a constant 2000ms assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL insertDTMF() with transceiver stopped in the middle should stop future tonechange events from firing assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL Calling insertDTMF() in the middle of tonechange events should cause future tonechanges to be updated to new tones assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL Calling insertDTMF() multiple times in the middle of tonechange events should cause future tonechanges to be updated the last provided tones assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
+FAIL Calling insertDTMF('') in the middle of tonechange events should stop future tonechange events from firing assert_unreached: Unexpected promise rejection: Waited too long for canInsertDTMF Reached unreachable code
 FAIL Setting transceiver.currentDirection to recvonly in the middle of tonechange events should stop future tonechange events from firing Cannot insert DTMF
 PASS Tone change event constructor works 
 PASS Tone change event with unexpected name should not crash

[Radar WebKit Bug Importer]

<rdar://problem/58841339>

[Truitt Savell]

This test appears to have been flaky for a long time, but got significantly more flaky around r253206.

[Truitt Savell]

Mac became more flakey around r253600

[youenn fablet]

Created attachment 388841 [details]
Patch

[youenn fablet]

Will upstream WPT changes separately

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 388841 [details]:

editing/spelling/spellcheck-async-remove-frame.html bug 158401 (authors: [e-mail], [e-mail], and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

The commit-queue encountered the following flaky tests while processing attachment 388841 [details]:

editing/spelling/spellcheck-attribute.html bug 206178 (authors: [e-mail], [e-mail], and [e-mail])
The commit-queue is continuing to process your patch.

[WebKit Commit Bot]

Comment on attachment 388841 [details]
Patch

Clearing flags on attachment: 388841

Committed r255166: <https://trac.webkit.org/changeset/255166>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.