Address PR feedback

Author: halter73

src/SignalR/common/Http.Connections/src/Internal/HttpConnectionContext.cs

@@ -443,11 +443,10 @@ public void MarkInactive()
         {
             lock (_stateLock)
             {
-                LastSeenUtc = DateTime.UtcNow;
-
                 if (Status == HttpConnectionStatus.Active)
                 {
                     Status = HttpConnectionStatus.Inactive;
+                    LastSeenUtc = DateTime.UtcNow;
                 }
             }
         }

src/SignalR/common/Http.Connections/src/Internal/HttpConnectionManager.cs

@@ -125,7 +125,7 @@ private async Task ExecuteTimerLoop()
                 {
                     try
                     {
-                        await ScanAsync();
+                        Scan();
                     }
                     catch (Exception ex)
                     {
@@ -137,7 +137,7 @@ private async Task ExecuteTimerLoop()
             Log.HeartBeatEnded(_logger);
         }
 
-        public Task ScanAsync()
+        public void Scan()
         {
             // Scan the registered connections looking for ones that have timed out
             foreach (var c in _connections)
@@ -164,8 +164,6 @@ public Task ScanAsync()
                     connection.TickHeartbeat();
                 }
             }
-
-            return Task.CompletedTask;
         }
 
         public void CloseConnections()

src/SignalR/common/Http.Connections/test/HttpConnectionDispatcherTests.cs

@@ -444,7 +444,7 @@ public async Task TransportEndingGracefullyWaitsOnApplicationLongPolling()
 
                     // The application is still running here because the poll is only killed
                     // by the heartbeat so we pretend to do a scan and this should force the application task to complete
-                    await manager.ScanAsync();
+                    manager.Scan();
 
                     // The application task should complete gracefully
                     await connection.ApplicationTask.OrTimeout();