Previously we have two different callsites here, one that passed in RawMessageInfo and one that passed in MessageInfo.

There's no reason to maintain the behavior of passing in MessageInfo. It simplifies the code to standardize on RawMessageInfo, and also helps avoid unnecessary computation.

I used this patch that adds a bunch of logs and ran a Release build of the iOS app pointed at production on my physical iOS device. I confirmed that the logs indicated a limited (minimal) amount of fetches for last updated time and MessagePreview, and that things generally looked okay in the app