Closed
Bug 593273
Opened 14 years ago
Closed 14 years ago
[D3D9] Crash when freeing ThebesLayerD3D9 after LayerManagerD3D9 [@ nsTArray<JSContext*>::IndexOf<JSContext*, nsDefaultComparator<JSContext*, JSContext*> >(JSContext* const&, unsigned int, nsDefaultComparator<JSContext*, JSContext*> const&) ]
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: mattwoodrow, Assigned: mattwoodrow)
References
Details
Attachments
(1 file)
|
(deleted),
patch
|
bas.schouten
:
review+
|
Details | Diff | Splinter Review |
When the addons compatibility check window pops up, I get a crash because the LayerManager is freed before it's layer.
|
Assignee | |
Comment 1•14 years ago
|
||
This fixes the crash since we don't need to remove the layers from the device.
May be better to look at why this is happening and delete the layers in advance.
I can try get a stack if necessary.
Attachment #471760 -
Flags: review?(bas.schouten)
|
||
Comment 2•14 years ago
|
||
Comment on attachment 471760 [details] [diff] [review]
Fix
I think this is actually okay for now, I can see why it's valid behavior since the layers don't hold a strong reference to their LayerManager (they shouldn't).
That list only exists to be able to reset the device, if the device is going down anyway, we don't care about it.
Attachment #471760 -
Flags: review?(bas.schouten) → review+
|
||
Updated•14 years ago
|
Assignee: nobody → matt.woodrow+bugzilla
Severity: normal → critical
Status: NEW → ASSIGNED
|
||
Updated•14 years ago
|
Blocks: d3d9-layers
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
|
||
Updated•14 years ago
|
Summary: [D3D9] Crash when freeing ThebesLayerD3D9 after LayerManagerD3D9 → [D3D9] Crash when freeing ThebesLayerD3D9 after LayerManagerD3D9 [@ nsTArray<JSContext*>::IndexOf<JSContext*, nsDefaultComparator<JSContext*, JSContext*> >(JSContext* const&, unsigned int, nsDefaultComparator<JSContext*, JSContext*> const&) ]
You need to log in
before you can comment on or make changes to this bug.
Description
•