Closed Bug 557671 Opened 11 years ago Closed 10 years ago
GL Image Layers need thread safety
YCbCr images should be threadsafe, SetData may be called on one thread while they are being drawn on another thread. Additionally SetCurrentImage and GetCurrentImage should lock while adding/releasing their references. We upload cairo surface image bits from setdata which should always be called on the main thread, so we don't need locking in there for now.
(In reply to comment #0) > Created an attachment (id=437443) [details] > Add locks to OGL image layers > > YCbCr images should be threadsafe, SetData may be called on one thread while > they are being drawn on another thread. This is actually not allowed. SetData must complete before you can call SetCurrentImage with that image.
So, remove the locking on the YCbCr images. Also, use mozilla::Monitor instead of PR_ stuff.
Processed roc's comments.
Remove two useless includes.
Attachment #437457 - Flags: review?(roc) → review+
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.