Add missing virtual destructors

[Zyrin]

🦟 Bug fix

Fixes #2829

Summary

Add missing virtual destructors so that "-Wnon-virtual-dtor" in combination with "-Werror" does not cause any issues.

Checklist

• Signed all commits for DCO
• Added tests
• Updated documentation (as needed)
• Updated migration guide (as needed)
• Consider updating Python bindings (if the library has them)
• codecheck passed (See contributing)
• All tests passed (See test coverage)
• While waiting for a review on your PR, please help review another open pull request to support the maintainers
• Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

[arjo129]

I would be in favour of this, however it is important to acknowledge that this potentially breaks ABI and should not be backported.

Gazebo is currently in a "feature freeze" till its next release, so we will not be able to merge these changes till after the jetty release as we will be holding main frozen for about 1.5 months.

[Zyrin]

Do you have any idea why the UtilTest.ResolveSdfWorldFile test fails on gz_sim-ci-pr_any-homebrew-amd64, but passes on gz_sim-ci-pr_any-noble-amd64?

[arjo129]

I don't think the test failures there are related to your change.

[azeey]

Gazebo is currently in a "feature freeze" till its next release, so we will not be able to merge these changes till after the jetty release as we will be holding main frozen for about 1.5 months.

I'd consider this a bug fix so it's okay to merge while we are in feature freeze.

[azeey]

Failing test is not related to this PR (see #3031)