Closed
Bug 1994891
Opened 6 months ago
Closed 5 months ago
Implement `transfer` for AudioDataInit
Categories
(Core :: Audio/Video: Web Codecs, task, P3)
Core
Audio/Video: Web Codecs
Tracking
()
RESOLVED
FIXED
146 Branch
| Tracking | Status | |
|---|---|---|
| firefox146 | --- | fixed |
People
(Reporter: chunmin, Assigned: pommicket)
References
Details
Attachments
(1 file)
Similar to Bug 1861769, we should add a transfer property to AudioDataInit's WebIDL and follow the latest AudioData constructor algorithm. This will allow users to transfer ownership of data from a TypedArray to the AudioData, avoiding unnecessary copies. (See Bug 1861769, comment 1 for more details and rationale.)
The expected outcome is that the implementation should pass the related WPTs in transfering.https.any.js.
Updated•5 months ago
|
Assignee: nobody → pommicket
Status: NEW → ASSIGNED
Pushed by aosmond@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/1bd71e921991
https://hg.mozilla.org/integration/autoland/rev/778fdda19db7
Implement `transfer` for AudioDataInit r=aosmond
Pushed by abutkovits@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/9ee6cbb5aad6
https://hg.mozilla.org/integration/autoland/rev/81db789ce6de
Revert "Bug 1994891 - Implement `transfer` for AudioDataInit r=aosmond" for causing Hazard failures at AudioData.cpp.
Comment 4•5 months ago
|
||
Backed out for causing Hazard failures at AudioData.cpp.
Backout link: https://hg-edge.mozilla.org/integration/autoland/rev/81db789ce6de5d2564c5a34ea97473fbb3f2e268
Flags: needinfo?(pommicket)
Pushed by aosmond@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/a946c644949e
https://hg.mozilla.org/integration/autoland/rev/ae4de1794ea6
Implement `transfer` for AudioDataInit r=aosmond
Comment 6•5 months ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
status-firefox146:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 146 Branch
Updated•5 months ago
|
Flags: needinfo?(pommicket)
Updated•5 months ago
|
QA Whiteboard: [qa-triage-done-c147/b146]
You need to log in
before you can comment on or make changes to this bug.
Description
•