Bug 234979

Summary:

https://brianpeiris.github.io/spatial-audio-test/?ramped stops playing after a few seconds

Product:

WebKit

Reporter:

Chris Dumez <cdumez>

Component:

Web Audio

Assignee:

Chris Dumez <cdumez>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

achristensen, brianpeiris, cdumez, darin, eric.carlson, ews-watchlist, ggaren, glenn, jer.noble, philipj, sergio, webkit-bug-importer, youennf

Priority:

Keywords:

InRadar

Version:

WebKit Nightly Build

Hardware:

Unspecified

OS:

Unspecified

See Also:

https://bugs.webkit.org/show_bug.cgi?id=227199

Attachments:

Description	Flags
WIP Patch	ews-feeder: commit-queue-
WIP Patch	none
Patch	none
Patch	none

Chris Dumez

Reported 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.

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 attachment proposed patch, testcase, etc.

Chris Dumez

Comment 1 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.

Chris Dumez

Comment 2 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.

Chris Dumez

Comment 3 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.

Brian Peiris

Comment 4 2022-01-07 17:44:49 PST

Thanks for looking into this so quickly!

Chris Dumez

Comment 5 2022-01-10 08:44:34 PST

Created attachment 448755 [details] WIP Patch

Chris Dumez

Comment 6 2022-01-10 09:50:47 PST

Created attachment 448761 [details] Patch