Watching debug logs I noticed that maxTimeLoaded was getting very close to, but not exactly equal to duration. I think we need to stop relying on the floating point calculations for the 'completed' case.
Created attachment 50261 [details] try to avoid the floating point comparison by using the fill status variable
LGTM!
Attachment 50261 [details] was posted by a committer and has review+, assigning to Gustavo Noronha Silva for commit.
Comment on attachment 50261 [details] try to avoid the floating point comparison by using the fill status variable Finally got this committed as r56465. Thanks!