It is correct, at least in that it matches the previous behavior (d->commitTransaction did the same). It was added to solve some test failures.

Handlers should be using specific signals (eventDeleted) rather than the commit signal, which is meant to be watched to see when an asynchronous operation finishes.