[wpt-sync] Sync PR 13123 - Add allowance for race in DTMF ontonechange events

RESOLVED FIXED in Firefox 64

Status

()

Product:
Core
Component:
WebRTC
Type:
enhancement
Importance:
P4
normal
Status:
RESOLVED FIXED
Opened:
9 months ago
Updated:
8 months ago

People

(Reporter: wptsync, Unassigned)

Tracking

(Depends on 1 bug)

Version:
unspecified
Target:
mozilla64
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox64 fixed)

Details

(Whiteboard: [wptsync downstream],
URL
)

Assignee

Description

•
9 months ago 
Sync web-platform-tests PR 13123 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/13123
Details from upstream follow.

Harald Alvestrand <hta@chromium.org> wrote:
>  Add allowance for race in DTMF ontonechange events
>  
>  This gives latitude for one DTMF tone event to be reported after a change
>  to the tone buffer. This may happen when the firing of ontonechange is
>  delayed for long enough to let the whole next tone play out.
>  
>  Bug: chromium:816475
>  Change-Id: Id91011600b61f43152c8fb896d72433ccb871c61
>  Reviewed-on: https://chromium-review.googlesource.com/1236337
>  WPT-Export-Revision: 983d6b61f3b45dea0752cd2f21ecaf86baeb6f5d
 Assignee

Updated

•
9 months ago
Component: web-platform-tests → WebRTC
Product: Testing → Core
 Assignee

Comment 6

•
9 months ago 
Ran 1 tests


Tests that are disabled for instability:
/webrtc/RTCDTMFSender-ontonechange.https.html

Comment 8

•
8 months ago 
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/ed08ae0756f2
[wpt PR 13123] - Make DTMF tone change more deterministic, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/758fc6b2f029
[wpt PR 13123] - Update wpt metadata, a=testonly

Comment 9

•
8 months ago 
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bf515d03bd66
[wpt PR 13123] - Make DTMF tone change more deterministic, a=testonly
https://hg.mozilla.org/integration/mozilla-inbound/rev/87535197cdb7
[wpt PR 13123] - Update wpt metadata, a=testonly
Depends on: 1420640

Comment 10

•
8 months ago
bugherder 
https://hg.mozilla.org/mozilla-central/rev/bf515d03bd66
https://hg.mozilla.org/mozilla-central/rev/87535197cdb7
Status: NEW → RESOLVED
Closed: 8 months ago
status-firefox64: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
You need to log in before you can comment on or make changes to this bug.