Closed Bug 715822 Opened 12 years ago Closed 12 years ago

OGL CleanupResources should cleanup Texture resources too

Categories

(Core :: Graphics, defect)

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

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla12

People

(Reporter: romaxa, Assigned: romaxa)

References

Details

Attachments

(2 files)

Cleanup texture resources
6.35 KB, patch
mattwoodrow
: review+
Details | Diff | Splinter Review
Minor modification for reftests (Null check for root layer)
9.41 KB, patch
Details | Diff | Splinter Review 
On maemo we found that it is better to simply cleanup LayerManagerOGL resources when browser goes to background, and call Initialize() again when it comes to foreground. (instead of LayerManager switch HW->SW which has some problems with ShadowLayers).

It works mostly fine, but we should also cleanup Texture based resources which are keeping reference to GLContext (invalid after CleanupResources call) and later might crash.
This should cleanup most of resources.
Assignee: nobody → romaxa
Status: NEW → ASSIGNED
Attachment #588605 - Flags: review?(matt.woodrow)
Attachment #588605 - Flags: review?(matt.woodrow) → review+
https://hg.mozilla.org/mozilla-central/rev/797896e68634 (merge from inbound)
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla12
Depends on: 793604
You need to log in before you can comment on or make changes to this bug.