This has two benefits:

1. Avoids calling await getMessageNotifyType twice for every message.
2. Exposes messageNotifyType to the callsite, which is necessary for a later diff.

Depends on D14334

Tested in combination with later diffs, to confirm that it's possible for some message types to not set a thread to unread