message_send: Remove unnecessary user_ids argument.

cfcbf58cd1 rightly removed the use of `user_ids` in
`render_markdown`, which in turn makes it unnecessary in
`render_incoming_message`.

Remove the unnecessary parameter from `render_incoming_message`.

zerver/actions/message_edit.py

@@ -983,7 +983,6 @@ def check_update_message(
         rendering_result = render_incoming_message(
             message,
             content,
-            user_info["message_user_ids"],
             user_profile.realm,
             mention_data=mention_data,
         )

zerver/actions/message_send.py

@@ -128,7 +128,6 @@ def create_mirror_user_if_needed(
 def render_incoming_message(
     message: Message,
     content: str,
-    user_ids: Set[int],
     realm: Realm,
     mention_data: Optional[MentionData] = None,
     url_embed_data: Optional[Dict[str, Optional[UrlEmbedData]]] = None,
@@ -528,7 +527,6 @@ def build_message_send_dict(
     rendering_result = render_incoming_message(
         message,
         message.content,
-        info["active_user_ids"],
         realm,
         mention_data=mention_data,
         email_gateway=email_gateway,

zerver/worker/queue_processors.py

@@ -884,11 +884,6 @@ class FetchLinksEmbedData(QueueProcessingWorker):
             if message.content is None:
                 return
 
-            query = UserMessage.objects.filter(
-                message=message.id,
-            )
-            message_user_ids = set(query.values_list("user_profile_id", flat=True))
-
             # Fetch the realm whose settings we're using for rendering
             realm = Realm.objects.get(id=event["message_realm_id"])
 
@@ -896,7 +891,6 @@ class FetchLinksEmbedData(QueueProcessingWorker):
             rendering_result = render_incoming_message(
                 message,
                 message.content,
-                message_user_ids,
                 realm,
                 url_embed_data=url_embed_data,
             )