compositor: Do not clear timeout on retry

When using vblank (either GLX or Xpresent), a previous paint request may
still be pending while new damage notifications are triggered by clients
updates.

If that occurs, xfwm4 does not cancel the existing timeout previously
created timeout (so it will retry automatically) but clears the timeout
id.

As a result, if new damage occurs, a new timeout would be created while
an existing one is still running.

When using Xpresent while the screen is turned off, no Xpresent event
notification occur, which increases the risk of the above occuring.

Clear the timeout id only when the repaint succeeded, so we don't pile
up multiple retries.

Signed-off-by: Olivier Fourdan <fourdan@xfce.org>
See-also: #502
2 jobs for master in 3 minutes and 10 seconds (queued for 1 second)
Status Job ID Name Coverage
  Build
passed #10221
build

00:01:20

 
  Distcheck
passed #10222
distcheck

00:01:49