RESOLVED FIXED 213360
Remove setVelocity() from PannerNode 
https://bugs.webkit.org/show_bug.cgi?id=213360
Summary Remove setVelocity() from PannerNode
Clark Wang
Reported 2020-06-18 15:43:40 PDT
Removed velocity from PannerNode
Attachments
Patch (5.49 KB, patch)
2020-06-18 17:11 PDT, Clark Wang 		no flags
DetailsFormatted DiffDiff
Patch (7.05 KB, patch)
2020-06-19 09:11 PDT, Clark Wang 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Clark Wang
Comment 1 2020-06-18 17:11:16 PDT
Created attachment 402253 [details] Patch
Chris Dumez
Comment 2 2020-06-18 17:15:22 PDT
Comment on attachment 402253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402253&action=review > Source/WebCore/ChangeLog:8 > + Re-baselined previous test that now passes with velocity removed. Changelog is missing an explanation of the change and a link to the relevant spec section.
Darin Adler
Comment 3 2020-06-18 17:24:47 PDT
Comment on attachment 402253 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=402253&action=review > Source/WebCore/Modules/webaudio/PannerNode.cpp:307 > + const FloatPoint3D& sourceVelocity = FloatPoint3D(0, 0, 0); This seems a peculiar way to write this. Not even sure it has defined behavior. I think the FloatPoint3D object may be destroyed and the end of the full expression? Maybe this should just be: const FloatPoint32 listenerVelocity; But of course, this is also a constant that’s always zero and so we could simplify the code below.
Clark Wang
Comment 4 2020-06-19 09:11:51 PDT
Created attachment 402289 [details] Patch
EWS
Comment 5 2020-06-19 17:53:46 PDT
Committed r263304: <https://trac.webkit.org/changeset/263304> All reviewed patches have been landed. Closing bug and clearing flags on attachment 402289 [details].
Radar WebKit Bug Importer
Comment 6 2020-06-19 17:54:34 PDT
<rdar://problem/64551041>
Note You need to log in before you can comment on or make changes to this bug.