Closed
Bug 1205557
Opened 9 years ago
Closed 9 years ago
MacIOSurfaceTextureClientOGL not properly implementing locking
Categories
(Core :: Graphics: Layers, defect)
Core
Graphics: Layers
Tracking
()
RESOLVED
WONTFIX
Tracking | Status | |
---|---|---|
firefox43 | --- | affected |
People
(Reporter: jya, Unassigned)
References
Details
(Whiteboard: [gfx-noted])
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Upon reviewing bug 1072313 ; this caught my attention.
There's no actual locking put in place with MacIOSurface ; and the logic to report if a surface is locked appears broken.
No idea if it's worth doing, but this will apply the same logic as what is done for other GPU surface (D3D9, D3D11 and others).
Reporter | ||
Comment 1•9 years ago
|
||
Attachment #8662194 -
Flags: review?(matt.woodrow)
Comment 2•9 years ago
|
||
Comment on attachment 8662194 [details] [diff] [review]
Implement proper MacIOSSurfaceTextureClientOGL locking.
Review of attachment 8662194 [details] [diff] [review]:
-----------------------------------------------------------------
We never call Lock() on a MacIOSurfaceTextureClientOGL, and MacIOSurface::Lock probably doesn't really do what we'd want anyway.
How about we just assert that this isn't called?
Attachment #8662194 -
Flags: review?(matt.woodrow)
It'd be nice for Lock()/Unlock() to return the status based on the io surface lock, rather than assume the lock succeeded in that case. Non-withstanding Matt's comment to assert, or even better, MOZ_CRASH.
Updated•9 years ago
|
Whiteboard: [gfx-noted]
Reporter | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•