Closed Bug 998113 Opened 11 years ago Closed 10 years ago

Stop creating a Thebes backed gfxContext in gfxCachedTempSurface::Get

Categories

(Core :: Graphics, defect)

Product:
Core
Component:
Graphics
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
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: 10 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.