Closed Bug 1288474 Opened 4 years ago Closed 4 years ago
Client Pool should actually stop once Destroy() has been called on it
I noticed some interesting behaviour that is non-obvious when I was changing the TextureClientPool's allocation strategy. On shutdown, we call TextureClientPool::Destroy(). This sets the pool size to 0, and drops all its refs to any TextureClients it owns. However, what is not obvious is that TextureClients are still returned after Destroy() has been called. This is "handled" by accepting them then calling ShrinkToMaximumSize() which then immediately discards them. We should probably just not accept them in the first place.
Assignee: nobody → gwright
Attachment #8773396 - Flags: review?(nical.bugzilla)
Attachment #8773396 - Flags: review?(nical.bugzilla) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/d91042a4ca77 Don't accept TextureClient returns if the pool has been destroyed r=nical
You need to log in before you can comment on or make changes to this bug.