Interrupt thread while decoding if coroutine is cancelled.

[colinrtwhite]

This allows for much faster cancellation as we can cancel in-flight decode operations.

Fixes #378

[Jawnnypoo]

Good stuff!

[Jawnnypoo]

Do we still need the deferred stuff too?

coil/coil-base/src/main/java/coil/RealImageLoader.kt

Line 253 in 9930c2a

deferred.invokeOnCompletion { throwable ->

[colinrtwhite]

Do we still need the deferred stuff too?

coil/coil-base/src/main/java/coil/RealImageLoader.kt

Line 253 in 9930c2a

deferred.invokeOnCompletion { throwable ->

Yep, invokeOnCompletion will still be called if the thread is interrupted. We also still need all the logic in invokeOnCompletion since it handles setting the error drawable, some bitmap pooling work, notifying listeners, etc.

[colinrtwhite]

Seeing a performance regression when testing this (related to runInterruptible not 1.3.6). Going to hold off merging for the moment and investigate.

[colinrtwhite]

I think the performance regression I was seeing on my Pixel 1 was a local config issue and haven't been able to reproduce. Going to merge this in and keep an eye on it.