Closed Bug 998113 Opened 7 years ago Closed 7 years ago

Stop creating a Thebes backed gfxContext in gfxCachedTempSurface::Get

Categories

(Core :: Graphics, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla33

People

(Reporter: jwatt, Assigned: jwatt)

References

Details

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?
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.
Assignee: nobody → jwatt
I fixed this in bug 1034528.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Depends on: 1034528
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.