Closed Bug 1289380 Opened 3 years ago Closed 3 years ago
Buffer Provider Shared
I shuffled things around a bit while investigating various copy-on-write canvas issues. I find the resulting code easier to understand, and it provides more control over when to remove the textures to avoid racing with layer transactions.
This makes PersistentBufferProviderShared store a vector of textures and use indices to tell which is the front and back buffers. It makes debugging easier by making the number of textures allocated explicit and makes sure we don't clear used textures from outside of a transaction. Some other small fixes got folded in by accident, they are probably not worth filing bugs for: * make sure we don't use PersistentBufferProviderShared with skiagl * make sure we don't accidentally pass the global error target to the buffer provider.
Attachment #8774699 - Flags: review?(edwin)
Attachment #8774699 - Flags: review?(edwin) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/fc5b4f286716 Simplify PersistentBufferProviderShared. r=edwin
You need to log in before you can comment on or make changes to this bug.