WebAudio biquad.html demo's canvas code OOMs Android

RESOLVED FIXED in Firefox 24

Status

()

defect
--
major
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: cpeterson, Unassigned)

Tracking

unspecified
mozilla24
All
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox23 affected, firefox24 fixed)

Details

()

Reporter

Description

6 years ago
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.
Reporter

Updated

6 years ago
Blocks: webaudio

Comment 1

6 years ago
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)
Reporter

Comment 2

6 years ago
(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
Reporter

Updated

6 years ago
No longer blocks: webaudio

Comment 3

6 years ago
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)

Comment 5

6 years ago
Chris, can you please test this again?
Flags: needinfo?(cpeterson)
Reporter

Comment 6

6 years ago
I tested today's Nightly build (2013-05-16) and the canvas leak seems to be fixed. :D
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Flags: needinfo?(cpeterson)
Resolution: --- → FIXED

Updated

6 years ago
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.