mth/ebiten
1
0
Fork 0
mirror of https://github.com/hajimehoshi/ebiten.git synced 2024-12-25 03:08:54 +01:00
Code Issues Packages Projects Releases Wiki Activity

audio: Bug fix: Touch is required by Chrome 55+ on Android

Browse Source
This commit is contained in:
Hajime Hoshi 2016-12-17 00:14:33 +09:00
parent 86966eb22f
commit d80b213184
1 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
audio/internal/driver/driver_js.go
View File

@ -40,6 +40,17 @@ func isIOS() bool {
return true return true
} }
func isAndroidChrome() bool {
ua := js.Global.Get("navigator").Get("userAgent").String()
if !strings.Contains(ua, "Android") {
return false
}
if !strings.Contains(ua, "Chrome") {
return false
}
return true
}
func NewPlayer(sampleRate, channelNum, bytesPerSample int) (*Player, error) { func NewPlayer(sampleRate, channelNum, bytesPerSample int) (*Player, error) {
class := js.Global.Get("AudioContext") class := js.Global.Get("AudioContext")
if class == js.Undefined { if class == js.Undefined {
@ -55,9 +66,12 @@ func NewPlayer(sampleRate, channelNum, bytesPerSample int) (*Player, error) {
bufferedData: []byte{}, bufferedData: []byte{},
context: class.New(), context: class.New(),
} }
// iOS requires touch event to use AudioContext. // iOS and Android Chrome requires touch event to use AudioContext.
if isIOS() { if isIOS() || isAndroidChrome() {
js.Global.Get("document").Call("addEventListener", "touchend", func() { js.Global.Get("document").Call("addEventListener", "touchend", func() {
// Resuming is necessary as of Chrome 55+ in some cases like different
// domain page in an iframe.
p.context.Call("resume")
p.context.Call("createBufferSource").Call("start", 0) p.context.Call("createBufferSource").Call("start", 0)
p.positionInSamples = int64(p.context.Get("currentTime").Float() * float64(p.sampleRate)) p.positionInSamples = int64(p.context.Get("currentTime").Float() * float64(p.sampleRate))
}) })