Open
Bug 1184057
Opened 9 years ago
Updated 2 years ago
Rewrite AudioEventTimeline.h
Categories
(Core :: Web Audio, task, P3)
Core
Web Audio
Tracking
()
NEW
People
(Reporter: padenot, Assigned: padenot)
References
(Blocks 7 open bugs)
Details
(Keywords: perf)
AudioEventTimeline is the code that handles AudioParam for Web Audio API.
It has a number of issues:
- It's way too complicated
- It has bugs (we don't support events of different types at the same time, SetTargetAtTime is buggy in a couple different ways)
- It's not fast enough, even with baku's optimizations
I think we should just rewrite it to be simpler, correct and faster. Additionally, it's a opportunity to implement the couple spec changes that happened in the area.
I'm taking this bug to work on it on the side (I've got a design ready), but feel free to take it as I won't get to it immediately.
Assignee | ||
Updated•9 years ago
|
Blocks: webaudioperf
Updated•9 years ago
|
Rank: 10
Priority: -- → P1
Updated•9 years ago
|
Blocks: webaudioperf_parity
Updated•9 years ago
|
No longer blocks: webaudioperf
Comment 1•8 years ago
|
||
Remaining from bug 1140448 comment 0:
"- We should determine, in the 128 ticks block, the boundaries for which to compute which event, and then compute the actual values in one pass after
- Then if really needed, we can speed it up with SIMD, since it's all linearly independent functions
Bonus points if we can make sure this code is independent enough of Gecko so that we can test it easily (as it is now), and nicely emscriptenable so that web app can do content-side (as opposed to audio thread-side) computations, as they often want to do: https://github.com/jsantell/web-audio-automation-timeline"
Updated•8 years ago
|
Blocks: webaudioperf
Comment 2•7 years ago
|
||
Mass change P1->P2 to align with new Mozilla triage process
Priority: P1 → P2
Comment 3•6 years ago
|
||
Moving to p3 because no activity for at least 1 year(s).
See https://github.com/mozilla/bug-handling/blob/master/policy/triage-bugzilla.md#how-do-you-triage for more information
Priority: P2 → P3
Updated•3 years ago
|
Type: defect → task
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•