Closed
Bug 1489487
Opened 7 years ago
Closed 7 years ago
[wpt-sync] Sync PR 12895 - Implement DTMF [[ToneBuffer]] in the blink layer
Categories
(Core :: WebRTC, enhancement, P4)
Core
WebRTC
Tracking
()
RESOLVED
FIXED
mozilla64
| Tracking | Status | |
|---|---|---|
| firefox64 | --- | fixed |
People
(Reporter: wpt-sync, Unassigned)
References
()
Details
(Whiteboard: [wptsync downstream])
Sync web-platform-tests PR 12895 into mozilla-central (this bug is closed when the sync is complete).
PR: https://github.com/web-platform-tests/wpt/pull/12895
Details from upstream follow.
Harald Alvestrand <hta@chromium.org> wrote:
> Implement DTMF [[ToneBuffer]] in the blink layer
>
> This CL makes the Blink layer keep a copy of the tone buffer
> and update it on insertDTMF and ontonechange events only; this
> makes it possible to expose the state of the tone buffer at the
> time the event is fired to the Javascript callback.
>
> It removes a queueing step inside the DTMF sender, because
> that queueing step destroyed the consistency.
>
> Bug: chromium:816475
> Change-Id: I5aa68396299a67d6cea1e8a17d364f553514c291
> Reviewed-on: https://chromium-review.googlesource.com/1213084
> WPT-Export-Revision: 7a601819ca51a2dc6af6014fca63f7837ed3f477
| Assignee | ||
Updated•7 years ago
|
Component: web-platform-tests → WebRTC
Product: Testing → Core
| Assignee | ||
Comment 1•7 years ago
|
||
| Assignee | ||
Comment 2•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=4dd9ebdffc56e7f4d412de4666558c8e23819b71
| Assignee | ||
Comment 3•7 years ago
|
||
| Assignee | ||
Comment 4•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=f8f2d209ae9ebe348f678f22ff3c82e4364648a8
| Assignee | ||
Comment 5•7 years ago
|
||
| Assignee | ||
Comment 6•7 years ago
|
||
| Assignee | ||
Comment 7•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=9ebe3a5a90e314466001c0cee7e4c09a883bede6
| Assignee | ||
Comment 8•7 years ago
|
||
| Assignee | ||
Comment 9•7 years ago
|
||
Ran 3 tests and 8 subtests
OK : 2
PASS : 7
FAIL : 1
Tests that are disabled for instability:
/webrtc/RTCDTMFSender-ontonechange.https.html
| Assignee | ||
Comment 10•7 years ago
|
||
Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=e3f0b695d255cef88606b3901b9e498f2f10f449
Comment 11•7 years ago
|
||
Pushed by wptsync@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/058710dfff1d
[wpt PR 12895] - Implement DTMF [[ToneBuffer]] in the blink layer, a=testonly
Comment 12•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years 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.
Description
•