Sure, but I'm thinking about the purpose of the abort method here. It makes sense that it would empty the arrays, and that it would invalidate the indexPaths so the image source isn't making pointless calls to its delegate as any lingering images loads complete. The way the class is written the calls will be ignored, but why make them at all?
The `ReaderDetailView` sets its `ReaderMediaQueue` delegate to nil, making the call superfluous, which is your point.  I'm not worried about this for a couple of reasons. One is we could potentially use the ReaderMediaQueue for other things where its delegate is still around. Another is that removing it would be making an assumption of what the delegate is doing and that doesn't seem good.
Thoughts?