recent_senders [nfc]: Make addAll take QueueList instead of List

This will avoid copying the list on each call to `addAll`.

Also make `removeAll` take `QueueList` instead of `List`.

Author: sm-sayedi

lib/model/recent_senders.dart

@@ -34,13 +34,13 @@ class RecentSenders {
   /// [messages] should be sorted by [id] ascendingly, which are, the way app
   /// receives and handles messages.
   void handleMessages(List<Message> messages) {
-    final messagesByUserInStream = <(int, int), List<int>>{};
-    final messagesByUserInTopic = <(int, String, int), List<int>>{};
+    final messagesByUserInStream = <(int, int), QueueList<int>>{};
+    final messagesByUserInTopic = <(int, String, int), QueueList<int>>{};
     for (final message in messages) {
       if (message is! StreamMessage) continue;
       final StreamMessage(:streamId, :topic, :senderId, id: int messageId) = message;
-      (messagesByUserInStream[(streamId, senderId)] ??= []).add(messageId);
-      (messagesByUserInTopic[(streamId, topic, senderId)] ??= []).add(messageId);
+      (messagesByUserInStream[(streamId, senderId)] ??= QueueList()).add(messageId);
+      (messagesByUserInTopic[(streamId, topic, senderId)] ??= QueueList()).add(messageId);
     }
 
     for (final entry in messagesByUserInStream.entries) {
@@ -70,11 +70,11 @@ class RecentSenders {
   void handleDeleteMessageEvent(DeleteMessageEvent event, Map<int, Message> cachedMessages) {
     if (event.messageType != MessageType.stream) return;
 
-    final messagesByUser = <int, List<int>>{};
+    final messagesByUser = <int, QueueList<int>>{};
     for (final id in event.messageIds) {
       final message = cachedMessages[id] as StreamMessage?;
       if (message == null) continue;
-      (messagesByUser[message.senderId] ??= []).add(id);
+      (messagesByUser[message.senderId] ??= QueueList()).add(id);
     }
 
     final DeleteMessageEvent(:streamId!, :topic!) = event;
@@ -128,15 +128,15 @@ class MessageIdTracker {
   /// Add the messages IDs to the tracker list at the proper place, if not present.
   ///
   /// [newIds] should be sorted ascendingly.
-  void addAll(List<int> newIds) {
+  void addAll(QueueList<int> newIds) {
     if (ids.isEmpty) {
-      ids = QueueList.from(newIds);
+      ids = newIds;
       return;
     }
     ids = setUnion(ids, newIds);
   }
 
-  void removeAll(List<int> idsToRemove) {
+  void removeAll(QueueList<int> idsToRemove) {
     ids.removeWhere((id) {
       final i = lowerBound(idsToRemove, id);
       return i < idsToRemove.length && idsToRemove[i] == id;