920239 - Tie SkiaGL texture lifetime to that of the bitmap

Status: RESOLVED WONTFIX

(Core :: Graphics: Canvas2D, defect)

Description

[James Willcox (:snorp) ([e-mail]) (he/him)]

Right now when you do something like drawImage() with a SkiaGL-backed canvas, we have some caching behavior that is less than ideal. First of all, we cache the moz2d SourceSurface ourselves. With SkiaGL, this contains a SkBitmap. Once you draw this, as drawImage() does, there is a texture in the Skia texture cache as well. Our SourceSurface cache appears to expire things after a period of time has elapsed. The Skia one is LRU. So once you hit the maximum texture cache size, it's going to stay there forever (at least for that GrContext/DrawTarget).

We should simply tie the texture to the bitmap. That way we really just have the one SourceSurface cache eating up memory.

Comment 1

[Darkspirit]

SkiaGL was removed by bug 1530471.