Simplify sendComplete: remove chatId param and dead refreshFragment logic

sendComplete is only ever called from the message-send path, never the
draft path, so the passed chatId always equals this.chatId. The
refreshFragment branch and this.chatId = chatId assignment were dead
code. Removing them also eliminates the class of bug where an old
chatId could accidentally revert the activity's current chat.

Co-authored-by: adbenitez <24558636+adbenitez@users.noreply.github.com>

src/main/java/org/thoughtcrime/securesms/ConversationActivity.java

@@ -1182,6 +1182,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
     playbackViewModel.stopNonMessageAudioPlayback();
 
     DcContext dcContext = DcHelper.getContext(context);
+    final int currentChatId = dcChat.getId();
     Util.runOnAnyBackgroundThread(() -> {
       DcMsg msg = null;
       int recompress = 0;
@@ -1191,9 +1192,9 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
         if (action == ACTION_SEND_OUT) {
           dcContext.sendEditRequest(msgId, body);
         } else {
-          dcContext.setDraft(chatId, null);
+          dcContext.setDraft(currentChatId, null);
         }
-        future.set(chatId);
+        future.set(currentChatId);
         return;
       }
 
@@ -1204,7 +1205,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
 
         try {
           if (slideDeck.getWebxdctDraftId() != 0) {
-            msg = dcContext.getDraft(chatId);
+            msg = dcContext.getDraft(currentChatId);
           } else {
             List<Attachment> attachments = slideDeck.asAttachments();
             for (Attachment attachment : attachments) {
@@ -1253,7 +1254,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
         // for WEBXDC, drafts are just sent out as is.
         // for preparations and other cases, cleanup draft soon.
         if (msg == null || msg.getType() != DcMsg.DC_MSG_WEBXDC) {
-          dcContext.setDraft(dcChat.getId(), null);
+          dcContext.setDraft(currentChatId, null);
         }
 
         if(msg!=null) {
@@ -1269,7 +1270,7 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
                                                    false
                                                    );
             });
-            doSend = VideoRecoder.prepareVideo(ConversationActivity.this, dcChat.getId(), msg);
+            doSend = VideoRecoder.prepareVideo(ConversationActivity.this, currentChatId, msg);
             Util.runOnMain(() -> {
               try {
                 if (progressDialog != null) progressDialog.dismiss();
@@ -1280,48 +1281,34 @@ public class ConversationActivity extends PassphraseRequiredActionBarActivity
           }
 
           if (doSend) {
-            if (dcContext.sendMsg(dcChat.getId(), msg) == 0) {
+            if (dcContext.sendMsg(currentChatId, msg) == 0) {
               String lastError = dcContext.getLastError();
               if (!"".equals(lastError)) {
                 Util.runOnMain(() -> Toast.makeText(ConversationActivity.this, lastError, Toast.LENGTH_LONG).show());
               }
-              future.set(chatId);
+              future.set(currentChatId);
               return;
             }
           }
 
-          Util.runOnMain(() -> sendComplete(dcChat.getId()));
+          Util.runOnMain(() -> sendComplete());
         }
       } else {
-        dcContext.setDraft(dcChat.getId(), msg);
+        dcContext.setDraft(currentChatId, msg);
       }
-      future.set(chatId);
+      future.set(currentChatId);
     });
 
     return future;
   }
 
 
-  protected void sendComplete(int chatId) {
-    boolean refreshFragment = (chatId != this.chatId);
-    this.chatId = chatId;
-
+  protected void sendComplete() {
     if (fragment == null || !fragment.isVisible() || isFinishing()) {
       return;
     }
 
     fragment.setLastSeen(-1);
-
-    if (refreshFragment) {
-      fragment.reload(recipient, chatId);
-      try {
-        int accId = rpc.getSelectedAccountId();
-        DcHelper.getNotificationCenter(this).updateVisibleChat(accId, chatId);
-      } catch (RpcException e) {
-        Log.e(TAG, "rpc.getSelectedAccountId() failed", e);
-      }
-    }
-
     fragment.scrollToBottom();
     attachmentManager.cleanup();
   }

src/main/java/org/thoughtcrime/securesms/components/InputPanel.java

@@ -132,10 +132,20 @@ public class InputPanel extends ConstraintLayout
       quoteAnimator.cancel();
     }
 
-    quoteAnimator =
-        createHeightAnimator(quoteView, originalHeight, this.quoteView.getMeasuredHeight(), null);
-
-    quoteAnimator.start();
+    int finalHeight = this.quoteView.getMeasuredHeight();
+    if (finalHeight > 0) {
+      quoteAnimator =
+          createHeightAnimator(quoteView, originalHeight, finalHeight, null);
+      quoteAnimator.start();
+    } else {
+      // Fallback when getMeasuredHeight() returns 0 (view not yet laid out).
+      // Setting WRAP_CONTENT ensures the quote is visible instead of animating to 0 height.
+      ViewGroup.LayoutParams params = quoteView.getLayoutParams();
+      if (params != null) {
+        params.height = ViewGroup.LayoutParams.WRAP_CONTENT;
+        quoteView.setLayoutParams(params);
+      }
+    }
   }
 
   public void clearQuoteWithoutAnimation() {