The test normally ends when the image is discarded. But there is nothing in the test that requires the image to ever be decoded. To properly test the original bug that this test is intended for we want the following sequence: imgRequestProxy::Cancel, imgRequestProxy::ChangeOwner, and then the image getting discarded. The cancel call makes it hard to get a full decode started, especially since it needs to happen before the ChangeOwner call which itself happens quite early. So it appears that there is no easy way to modify the test to keep this sequence of events 100% of the time.
However the test as it currently stands does indeed get this sequence of events most of the time, so it continues to test the right thing most of the time. The few times it doesn't test the right thing we can just move on.
