Clean up ahead of PR.

// FREEBIE

Signal/src/ViewControllers/ConversationView/MessagesViewController.m

@@ -558,7 +558,6 @@ typedef enum : NSUInteger {
                                                                userInfo:nil
                                                                 repeats:NO];
 
-    // TODO: Why are we marking as read here? Shouldn't our repeating 1-sec read timer be sufficient?
     [self markVisibleMessagesAsRead];
 }
 
@@ -768,7 +767,7 @@ typedef enum : NSUInteger {
 - (void)startReadTimer
 {
     [self.readTimer invalidate];
-    self.readTimer = [NSTimer weakScheduledTimerWithTimeInterval:2
+    self.readTimer = [NSTimer weakScheduledTimerWithTimeInterval:3.f
                                                           target:self
                                                         selector:@selector(readTimerDidFire)
                                                         userInfo:nil
@@ -823,6 +822,7 @@ typedef enum : NSUInteger {
 
     [self cancelReadTimer];
     [self saveDraft];
+    [self markVisibleMessagesAsRead];
 
     [self cancelVoiceMemo];