Closed Bug 872208 Opened 12 years ago Closed 12 years ago

WebAudio biquad.html demo's canvas code OOMs Android

Categories

(Core :: Graphics: Canvas2D, defect)

All
Android
defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla24
Tracking Status
firefox23 --- affected
firefox24 --- fixed

People

(Reporter: cpeterson, Unassigned)

References

()

Details

STR: 1. Load https://people.mozilla.com/~eakhgari/webaudio/biquad.html 2. Press the "Play" button 3. Wait ~10 seconds while watching adb logcat RESULT: Android will crash with OOM exceptions in GL or DB code.
Blocks: webaudio
Do you have a stack? What if you remove the canvas stuff in there? (The repo is here https://github.com/ehsan/webaudio-tests)
Flags: needinfo?(cpeterson)
(In reply to :Ehsan Akhgari (needinfo? me!) from comment #1) > Do you have a stack? What if you remove the canvas stuff in there? (The > repo is here https://github.com/ehsan/webaudio-tests) The stacks are not useful because they are random Java exceptions. This looks like a canvas leak, not a Web Audio leak. If I comment out biquad.html's canvas code, then the leaks "go away": https://people.mozilla.com/~cpeterson/webaudio/biquad.html
Component: Video/Audio → Canvas: 2D
Flags: needinfo?(cpeterson)
Summary: WebAudio biquad.html demo OOMs Android → WebAudio biquad.html demo's canvas code OOMs Android
No longer blocks: webaudio
That's terrible. This page paints three canvases on every refresh driver tick... This could be the kind of bug which would affects canvas 2D games. Vlad, have we seen this on such games on Android?
Flags: needinfo?(vladimir)
I'm pretty sure we had a recent canvas leak on mobile, but I think it's been fixed in the previous day or two..
Flags: needinfo?(vladimir)
Chris, can you please test this again?
Flags: needinfo?(cpeterson)
I tested today's Nightly build (2013-05-16) and the canvas leak seems to be fixed. :D
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: needinfo?(cpeterson)
Resolution: --- → FIXED
Depends on: 871390
No longer depends on: 870165
Target Milestone: --- → mozilla24
You need to log in before you can comment on or make changes to this bug.