Aman Agrawal 459e1fc218 sent_messages: Pass correct reference of this in setTimeout. ...

We explicitly bind `this` to MessageState class which otherwise
was defaulting to `window`.

This resulted in variables like `this.received` and `this.local_id`
being incorrectly interpreted by called function
as `window.(received | local_id)` which are `undefined`.
Hence, frontend thinks that the message was never received.

It was noticed since this was the common log message when
a double message send bug was observed. This change in no
was indicates fixing of the double send bug, but is hopefully
one step forward in that direction.

2021-01-27 09:52:42 -08:00

3.9 KiB

Raw Blame History

View Raw