Summary: | REGRESSION (iOS 17.5 beta): Memory leak with audio sprites | ||
---|---|---|---|
Product: | WebKit | Reporter: | Matthew Hutchinson <matthew.hutchinson> |
Component: | Web Audio | Assignee: | Nobody <webkit-unassigned> |
Status: | RESOLVED DUPLICATE | ||
Severity: | Blocker | CC: | ahmad.saleem792, cdumez, eric.carlson, jean-yves.avenard, jer.noble, kaikai0428, matthew.hutchinson, mattwindwer, webkit-bug-importer |
Priority: | P1 | Keywords: | InRadar |
Version: | Safari 17 | ||
Hardware: | iPhone / iPad | ||
OS: | iOS 17 | ||
URL: | https://www.macguffinandshemp.com/iOS17test/ | ||
Attachments: |
Description
Matthew Hutchinson
2024-04-21 06:32:50 PDT
Created attachment 471031 [details]
memory profile iOS 17.4.1 - this does not result in the tab resetting
This is the same page running on iOS17.4.1 As you can see the memory stays fairly flat.
Updated https://www.macguffinandshemp.com/iOS17test/ to generate audio based on: https://developer.mozilla.org/en-US/docs/Web/API/BaseAudioContext/createBufferSource If you start a "white noise loop" you can see from the attachment that after 60 seconds iOS 17.4.1 memory = 199.82Mb iOS 17.5 memory = 496Mb Created attachment 471032 [details]
comparison of memory usage in iOS17.4.1 vs iOS17.5
This issue effects both Safari and Chrome on iOS 17.5 Test crashing (from GitHub) - https://rayji0428.github.io/demo/cocos/index.html ^ haven't test myself. NOTE - Please mute or reduce volume - it is not pleasing sound at all. I suspect I fixed very recently in Bug 272568. (In reply to Ahmad Saleem from comment #5) > Test crashing (from GitHub) - > https://rayji0428.github.io/demo/cocos/index.html > > ^ haven't test myself. > > NOTE - Please mute or reduce volume - it is not pleasing sound at all. Please use this one to test. Sorry about the volume, just trying to make it crash sooner... https://rayji0428.github.io/demo/howler/index.html (In reply to RayJi from comment #8) > (In reply to Ahmad Saleem from comment #5) > > Test crashing (from GitHub) - > > https://rayji0428.github.io/demo/cocos/index.html > > > > ^ haven't test myself. > > > > NOTE - Please mute or reduce volume - it is not pleasing sound at all. > > Please use this one to test. Sorry about the volume, just trying to make it > crash sooner... > https://rayji0428.github.io/demo/howler/index.html Thanks for the test case. I have confirmed that the jetsam reproduces before the fix for Bug 272568 but not after. This is a dupe. This issue should go away in a later build. *** This bug has been marked as a duplicate of bug 272568 *** (In reply to Chris Dumez from comment #9) > (In reply to RayJi from comment #8) > > (In reply to Ahmad Saleem from comment #5) > > > Test crashing (from GitHub) - > > > https://rayji0428.github.io/demo/cocos/index.html > > > > > > ^ haven't test myself. > > > > > > NOTE - Please mute or reduce volume - it is not pleasing sound at all. > > > > Please use this one to test. Sorry about the volume, just trying to make it > > crash sooner... > > https://rayji0428.github.io/demo/howler/index.html > > Thanks for the test case. I have confirmed that the jetsam reproduces before > the fix for Bug 272568 but not after. This is a dupe. This issue should go > away in a later build. > > *** This bug has been marked as a duplicate of bug 272568 *** This was indeed a regression in iOS 17.5 beta and we reverted the patch that caused it. Does this mean that the issue will not occur when iOS17.5 is releases officially? Thanks for the info Matt That's the plan! Can confirm this fixed in 17.5 beta 3 - thanks all Hi all, This issue is back on iOS 17.6 beta. I reproduced in Safari on my iPad Mini 5th gen by visiting https://www.macguffinandshemp.com/iOS17test/ (thank you for that!). Press "audio sprite loop." It will crash Safari sooner or later. It does not crash iOS 17.5.1. I'm pretty sure this is the same issue I reported last week - bug 275852. I verified that the fix is present in iOS 17.6 beta source code, we didn't accidentally lose it. We've been unable to reproduce bug 275852 so far, so if anyone is seeing similar issues in 17.6 beta, please comment there or file new bugs. Hi I've just tried my https://www.macguffinandshemp.com/iOS17test/ test on 17.6 (21G5052e). If I click the "audio sprite loop" button once it freezes Safari after around 200 loops. This also affects Chrome on the same device. FYI 17.5.1 is not freezing so far +1000 loops and still going. @Matthew - I think this bug is fixed because I tried following test case (you provided prior): Test Case: https://rayji0428.github.io/demo/howler/index.html I tried with 6600 repeat on iOS 17.5 and iOS 17.6 Beta and they behaved same. I think bug 275852 is different from this and at least, imo this is not back. But if you think it is reproducible, appreciate if you can open newer bug with test case, where you are able to reproduce and steps to reproduce, so we can try on our end. (In reply to Ahmad Saleem from comment #18) > @Matthew - I think this bug is fixed because I tried following test case > (you provided prior): > > Test Case: https://rayji0428.github.io/demo/howler/index.html > > I tried with 6600 repeat on iOS 17.5 and iOS 17.6 Beta and they behaved > same. I think bug 275852 is different from this and at least, imo this is > not back. > > But if you think it is reproducible, appreciate if you can open newer bug > with test case, where you are able to reproduce and steps to reproduce, so > we can try on our end. Sorry - test case is from someone else but again it is not reproducing while it was earlier reproducing for me. Hi @ahmad I agree that this is not the same as the original bug as there is no obvious increase in memory. The test https://www.macguffinandshemp.com/iOS17test/ does appear to freeze the browser when clicking bottom left button - audio sprite loop. After about 200-400 or so loops the browser will freeze. This is not happening in 17.5. I will open a new bug but if you were able to confirm that you see this behaviour that would be useful. Thanks (In reply to Matthew Hutchinson from comment #20) > Hi @ahmad I agree that this is not the same as the original bug as there is > no obvious increase in memory. > > The test https://www.macguffinandshemp.com/iOS17test/ does appear to freeze > the browser when clicking bottom left button - audio sprite loop. > After about 200-400 or so loops the browser will freeze. This is not > happening in 17.5. > > I will open a new bug but if you were able to confirm that you see this > behaviour that would be useful. > Thanks That's what I did in bug 275852. Reference - https://bugs.webkit.org/show_bug.cgi?id=275852#c12 |