executor: broadcast while locked

It's legal to do it after the lock but we always do it while locked, as most (if not all) implementations handle this specific case.

Should not be related to #28508 (closed). If the test failure stop with this commit, it means we have an issue in our condition implementation.