Fix autorepeat for game_controller_node

[abrandemuehl]

The game_controller_node event loop runs the logic to run the autorepeat logic only when no events are received on the SDL_WaitEventTimeout. This means that when repeated events occur such as holding the joystick in the same location or ignored events like deadzone axis events are received, the autorepeat logic is skipped.

This commit refactors the autorepeat and coalescing logic

1. Timeouts based on how long there should be until the next expected publish, whether that's the autorepeat interval or the coalesce period.
2. Unconditional checking of the timing for coalesced and autorepeat publishing.

This should fix the problem of the joy messages not properly autorepeating

[abrandemuehl]

@clalancette I wrote this before seeing #261, If you'd like I can check if it's possible to apply the same simpler fix. I think what I've done is correct but I'm a bit worried that I can't really test it against any sort of mock SDL event loop