Skip to content
This repository has been archived by the owner on Nov 4, 2023. It is now read-only.

AudioBugfix for Samsung/Android Browser #92

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

AudioBugfix for Samsung/Android Browser #92

wants to merge 1 commit into from

Conversation

JochenHeizmann
Copy link

On Samsung Devices (on my Galaxy S4Mini) playing a sound effect
sometimes results in a crash, throwing the following Exception:

InvalidStateError: DOM Exception 11

I've found out that it could happen that chan.waSource.stop( 0 )
throws this exception. However, as far as I understand the source,
this happens if stop is called twice or when waSource Object
isn't playing. So this if-branch shouldn't be reached, but it does.

I've fixed it by catching the exception and ignoring it, which is
fine in this case.

I'm sure there must be a more cleaner solution, but I wasn't
able to find one. This fix worked fine for me.

On Samsung Devices (on my Galaxy S4Mini) playing a sound effect
sometimes results in a crash, throwing the following Exception:

InvalidStateError: DOM Exception 11

I've found out that it could happen that chan.waSource.stop( 0 )
throws this exception. However, as far as I understand the source,
this happens if stop is called twice or when waSource Object
isn't playing. So this if-branch shouldn't be reached, but it does.

I've fixed it by catching the exception and ignoring it, which is
fine in this case.

I'm sure there must be a more cleaner solution, but I wasn't
able to find one. This fix worked fine for me.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant