Set looping on OGGVorbis default import options to false

[MichaelShaulskiy]

This was a cause of confusion for me when I was participating in a game jam.
Although I had disabled looping on the AudioStreamPlayer2D node, this change had no effect.

I discovered that the OGG import settings where to blame which have looping turned on by default.

First off all, this option shouldn't even exist. Looping should be controlled by the AudioStreamPlayer only and not be inherent to a resource.
Me and according to google many others had encountered a "bug" where a sound, that shouldn't be looping was looping despite looping being turned of in the Node.

Instead of removing it outright right now, maybe it should be turned off by default to not break any projects.

Bugsquad edit: This closes godotengine/godot-proposals#3120.

[Calinou]

Instead of removing it outright right now, maybe it should be turned off by default to not break any projects.

This default setting change will still break projects that rely on the existing behavior. Therefore, it can only be considered for 4.0. I would recommend opening a proposal for this too, so we can see whether this gathers sufficient community support.

The reason OGG files loop by default is that they're generally used for background music and ambient sounds, which are expected to be looped. For sound effects, it's better to use WAV files as they use up less CPU resources when played back. On the other hand, it's unlikely that WAV is used for background music or ambient sounds due to the high file size. This is why looping is disabled for WAV files by default.

[reduz]

I think given music looping is more involved and will require the BPM setting, we should set this flag by default to off now.
Feel free to rebase this, also should be applied to the mp3 importer.

[akien-mga]

Superseded by #71858.