Bug 234979 - https://brianpeiris.github.io/spatial-audio-test/?ramped stops playing after a few seconds
Summary: https://brianpeiris.github.io/spatial-audio-test/?ramped stops playing after ...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Audio (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Chris Dumez
URL:
Keywords: InRadar
Depends on:
Blocks:
 
Reported: 2022-01-07 12:57 PST by Chris Dumez
Modified: 2022-01-10 13:47 PST (History)
13 users (show)

See Also:

Attachments
WIP Patch (3.25 KB, patch)
2022-01-07 17:33 PST, Chris Dumez 		ews-feeder: commit-queue-
Details | Formatted Diff | Diff
WIP Patch (1.87 KB, patch)
2022-01-10 08:44 PST, Chris Dumez 		no flags Details | Formatted Diff | Diff
Patch (6.26 KB, patch)
2022-01-10 09:50 PST, Chris Dumez 		no flags Details | Formatted Diff | Diff
Patch (6.20 KB, patch)
2022-01-10 09:56 PST, Chris Dumez 		no flags Details | Formatted Diff | Diff
Show Obsolete (3) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Dumez 2022-01-07 12:57:12 PST 
https://brianpeiris.github.io/spatial-audio-test/?ramped stops playing after a few seconds in Safari but plays fine in Chrome.
Comment 1 Chris Dumez 2022-01-07 16:56:34 PST 
Doing a linear ramp of positionX / positionZ without doing a linear ramp of orientationX / orientationZ seems to work. This seems to have to do with the linear ramp of the orientation somehow.
Comment 2 Chris Dumez 2022-01-07 17:00:30 PST 
(In reply to Chris Dumez from comment #1)
> Doing a linear ramp of positionX / positionZ without doing a linear ramp of
> orientationX / orientationZ seems to work. This seems to have to do with the
> linear ramp of the orientation somehow.

In particular, it seems to be the linear ramp on the orientationZ that seems to be causing trouble here.
Comment 3 Chris Dumez 2022-01-07 17:33:23 PST 
Created attachment 448650 [details]
WIP Patch

This seems to fix it. I'll do more investigation and write a test next week.
Comment 4 Brian Peiris 2022-01-07 17:44:49 PST 
Thanks for looking into this so quickly!
Comment 5 Chris Dumez 2022-01-10 08:44:34 PST 
Created attachment 448755 [details]
WIP Patch
Comment 6 Chris Dumez 2022-01-10 09:50:47 PST 
Created attachment 448761 [details]
Patch
Comment 7 Chris Dumez 2022-01-10 09:56:09 PST 
Created attachment 448762 [details]
Patch
Comment 8 EWS 2022-01-10 13:46:12 PST 
Committed r287854 (245900@main): <https://commits.webkit.org/245900@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 448762 [details].
Comment 9 Radar WebKit Bug Importer 2022-01-10 13:47:19 PST 
<rdar://problem/87360182>