Following MDL-37641, this method is far from being efficient as each candidate file name is checked upon the draft area. It would be possible to create a method in file_storage which would return an unused file name using only one single query.

Also noting that repository::draftfile_exists() could certainly be part of the file_storage too and improved using a single query instead of fetching the whole stored_file object.