Prevent AudioBufferSourceNode.buffer to be set more than once
Categories
(Core :: Web Audio, defect, P3)
Tracking
()
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: padenot, Assigned: u538564)
Details
(Keywords: dev-doc-needed, Whiteboard: [spec])
Attachments
(3 files, 4 obsolete files)
6.03 KB,
patch
|
ehsan.akhgari
:
review-
|
Details | Diff | Splinter Review |
4.08 KB,
patch
|
Details | Diff | Splinter Review | |
47 bytes,
text/x-phabricator-request
|
Details | Review |
Reporter | ||
Comment 3•10 years ago
|
||
Reporter | ||
Comment 5•10 years ago
|
||
Reporter | ||
Comment 6•10 years ago
|
||
Reporter | ||
Updated•10 years ago
|
Reporter | ||
Updated•10 years ago
|
Comment 7•10 years ago
|
||
Reporter | ||
Comment 8•10 years ago
|
||
Comment 9•10 years ago
|
||
Reporter | ||
Comment 10•10 years ago
|
||
Comment 11•10 years ago
|
||
Reporter | ||
Comment 12•10 years ago
|
||
Comment 13•10 years ago
|
||
Reporter | ||
Comment 14•10 years ago
|
||
Assignee | ||
Comment 15•10 years ago
|
||
Reporter | ||
Comment 16•10 years ago
|
||
Assignee | ||
Comment 17•10 years ago
|
||
Reporter | ||
Comment 18•10 years ago
|
||
Comment 19•10 years ago
|
||
Assignee | ||
Comment 20•10 years ago
|
||
Reporter | ||
Comment 21•10 years ago
|
||
Reporter | ||
Comment 22•10 years ago
|
||
Comment 23•10 years ago
|
||
Reporter | ||
Comment 24•10 years ago
|
||
Reporter | ||
Comment 25•10 years ago
|
||
Comment 26•9 years ago
|
||
Updated•9 years ago
|
Updated•9 years ago
|
Reporter | ||
Comment 27•9 years ago
|
||
Comment 28•7 years ago
|
||
Reporter | ||
Comment 29•5 years ago
|
||
Ok so we've decided a while back what to do: it's allowed to set those attributes it to null
. I'm going to rebase https://bugzilla.mozilla.org/attachment.cgi?id=8613521 and land, since I didn't write it and it implements the correct behaviour.
Reporter | ||
Comment 30•5 years ago
|
||
Reporter | ||
Comment 31•5 years ago
|
||
(In reply to Paul Adenot (:padenot) from comment #29)
Ok so we've decided a while back what to do: it's allowed to set those attributes it to
null
. I'm going to rebase https://bugzilla.mozilla.org/attachment.cgi?id=8613521 and land, since I didn't write it and it implements the correct behaviour.
This was in fact missing an edge case.
Updated•5 years ago
|
Updated•5 years ago
|
Updated•5 years ago
|
Comment 32•5 years ago
|
||
Comment 33•5 years ago
|
||
Backed out changeset e3b626560b0d (bug 1161025) for causing crashtest failures on 966636.html.
Backout revision https://hg.mozilla.org/integration/autoland/rev/33d4b89801dabda575589825f3aa9b4bb6c07818
Failure log https://treeherder.mozilla.org/logviewer.html#?job_id=287294351&repo=autoland
Paul can you please take a look?
Updated•5 years ago
|
Updated•5 years ago
|
Comment 34•5 years ago
|
||
Comment 35•5 years ago
|
||
bugherder |
Reporter | ||
Updated•5 years ago
|
Description
•