Fix one variant of a scroll jump that occurs when decrypting an m.text

[lukebarnard1]

This calls onWidgetLoad after an event has been decrypted or a device verification has changed. It isn't called after a forceUpdate() in _onDecrypted but empirically the change reduces scroll jumps in e2e rooms.

[richvdh]

hrm. Does verification actually ever cause a resize? I'm thinking not, because the dom node is allocated and sized at initial render.

Should this not therefore be happening in _onDecrypted (or rather, on the callback of forceUpdate)?

(in case you missed it: _verifyEvent is asynchronous)

[richvdh]

shouldn't this be in the setState callback, so it happens after the content is rendered?

[lukebarnard1]

Well it changes the classes on the event tile so I'd rather just assume that it does cause a change.

It's true that it should be in the callback of setState though despite my previous optimistic empiricism.

[richvdh]

ok well in that case stick comments in _onDecrypted to say that the onWidgetLoad is happening in _verifyEvent. (And possibly one in _verifyEvent to say that it is covering the decryption.)

[richvdh]

lgtm