### Description

While working on the trace viewer I noticed that each `ack` causes
another message to be sent, which results in another `ack` so it keeps
looping.

This ensures `ack` only sends for the update case. Other messages like
sending a new viewrect will still send an update correctly with these
changes.

<!--
  ✍️ Write a short summary of your work.
  If necessary, include relevant screenshots.
-->

### Testing Instructions

<!--
  Give a quick description of steps to test your changes.
-->