@jmarshallnz : Because afaik this block is only here for streams without an actual filename (/extension). http will be checked once again down below by CUtil::SupportsWriteFileOperations(strCurrentFilePath)). By changing this at least items like http://server/movie.mp4 get their subtitle stored in the custom subtitle folder as eg. movie.srt so it can be reused the next time a user plays the video.

Btw. I think we should drop the http check altogether here and only check the extension.