Make sure timeout callbacks always get cleaned up (#831)

Under certain circumstances the `TimerHandle` used for query timeouts
does not get cleaned up in a timely fashion resulting in a temporary
(`timeout` seconds) memory leak.

Fixes: #830

Author: elprans

asyncpg/protocol/protocol.pyx

@@ -647,12 +647,12 @@ cdef class BaseProtocol(CoreProtocol):
         self.waiter.set_exception(asyncio.TimeoutError())
 
     def _on_waiter_completed(self, fut):
+        if self.timeout_handle:
+            self.timeout_handle.cancel()
+            self.timeout_handle = None
         if fut is not self.waiter or self.cancel_waiter is not None:
             return
         if fut.cancelled():
-            if self.timeout_handle:
-                self.timeout_handle.cancel()
-                self.timeout_handle = None
             self._request_cancel()
 
     def _create_future_fallback(self):