Closed
Bug 966867
Opened 11 years ago
Closed 11 years ago
Increasing delay in web audio output on systems with native sample rate > 44100
Categories
(Core :: Web Audio, defect, P1)
Tracking
()
RESOLVED
FIXED
mozilla29
| Tracking | Status | |
|---|---|---|
| firefox28 | --- | unaffected |
| firefox29 | --- | fixed |
People
(Reporter: karlt, Assigned: karlt)
References
Details
(Keywords: regression)
Attachments
(3 files)
|
(deleted),
text/html
|
Details | |
|
(deleted),
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
|
(deleted),
patch
|
kinetik
:
review+
|
Details | Diff | Splinter Review |
On a system with native audio sample rate of 48000:
1. Start browser.
2. load testcase.
3. click "Beep".
4. wait a few seconds.
5. click "Beep" again.
ER: similar response time for each click.
AR: response times are increasing.
I don't know why PA doesn't always return the same rate on my system, but sometimes it says 48000, and I see this bug.
|
Assignee | |
Comment 1•11 years ago
|
||
Regression from https://hg.mozilla.org/integration/mozilla-inbound/rev/bc103864507b
Compare https://hg.mozilla.org/integration/mozilla-inbound/rev/fa872452b89d
to see the bug.
Keywords: regression
|
|
Assignee | |
Updated•11 years ago
|
|
|
Assignee | |
Comment 2•11 years ago
|
||
A debug build catches this as:
###!!! ASSERTION: Something went wrong with rounding to block boundaries: 't == aTo', file /home/karl/moz/dev/content/media/MediaStreamGraph.cpp, line 1104
Assertion failure: n->SampleRate() == nextStream->SampleRate() (All AudioNodeStreams in the graph must have the same sampling rate), at /home/karl/moz/dev/content/media/MediaStreamGraph.cpp:1222
|
|
Assignee | |
Comment 3•11 years ago
|
||
Attachment #8369302 -
Flags: review?(kinetik)
|
|
Assignee | |
Updated•11 years ago
|
Priority: -- → P1
|
|
Assignee | |
Comment 4•11 years ago
|
||
Attachment #8369304 -
Flags: review?(kinetik)
|
||
Updated•11 years ago
|
Attachment #8369302 -
Flags: review?(kinetik) → review+
|
|
||
Updated•11 years ago
|
Attachment #8369304 -
Flags: review?(kinetik) → review+
|
|
Assignee | |
Comment 5•11 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/ce582b4aa4a1
https://hg.mozilla.org/integration/mozilla-inbound/rev/db60f1d540c4
Flags: in-testsuite-
|
||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/ce582b4aa4a1
https://hg.mozilla.org/mozilla-central/rev/db60f1d540c4
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla30
|
|
||
Comment 7•11 years ago
|
||
correcting target milestone, sorry for the bugmail spam
Target Milestone: mozilla30 → mozilla29
|
|
Assignee | |
Updated•11 years ago
|
tracking-firefox29:
? → ---
|
||
Updated•11 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•