998113 - Stop creating a Thebes backed gfxContext in gfxCachedTempSurface::Get

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[Jonathan Watt [:jwatt]]

One of the sources of Thebes backed gfxContexts for SVG is the one in gfxCachedTempSurface::Get. The only place that uses gfxCachedTempSurface is BasicLayerManager::mCachedSurface, so getting rid of this source may involve getting rid of that member as a sub-part of bug 989858. Matt, would you be able to take a look at this? Or is this essentially a dup of bug 989858?

Comment 1

[Matt Woodrow (:mattwoodrow)]

This is basically dead code right now since we don't use the cached surface if the gfxContext is backed by a DT (always).

http://mxr.mozilla.org/mozilla-central/source/gfx/layers/basic/BasicLayerManager.cpp#298

We might want this back, but it probably needs to take Bas' PushGroup API ideas into account.

I can't see how this is blocking you anyway.

Comment 2

[Jonathan Watt [:jwatt]]

I fixed this in bug 1034528.