Refactored ApiWrap::editMedia.

- Slightly refactored checkEntitiesAndViewsUpdate.
 - Slightly refactored HistoryMessage.

Telegram/SourceFiles/apiwrap.cpp

@@ -4472,22 +4472,19 @@ void ApiWrap::editMedia(
 		SendMediaType type,
 		TextWithTags &&caption,
 		const SendOptions &options) {
-	LOG(("EDIT MEDIA WITH TEXT %1").arg(caption.text));
+	if (list.files.empty()) return;
+
+	auto &file = list.files.front();
 	const auto to = fileLoadTaskOptions(options);
-	auto tasks = std::vector<std::unique_ptr<Task>>();
-	tasks.reserve(list.files.size());
-	for (auto &file : list.files) {
-		tasks.push_back(std::make_unique<FileLoadTask>(
-			file.path,
-			file.content,
-			std::move(file.information),
-			type,
-			to,
-			caption,
-			nullptr,
-			true));
-	}
-	_fileLoader->addTasks(std::move(tasks));
+	_fileLoader->addTask(std::make_unique<FileLoadTask>(
+		file.path,
+		file.content,
+		std::move(file.information),
+		type,
+		to,
+		caption,
+		nullptr,
+		true));
 }
 
 void ApiWrap::sendFiles(

Telegram/SourceFiles/app.cpp

@@ -139,7 +139,6 @@ namespace App {
 			auto entities = m.has_entities()
 				? TextUtilities::EntitiesFromMTP(m.ventities.v)
 				: EntitiesInText();
-			const auto media = m.has_media() ? &m.vmedia : nullptr;
 			existing->setText({ text, entities });
 			existing->updateSentMedia(m.has_media() ? &m.vmedia : nullptr);
 			existing->updateReplyMarkup(m.has_reply_markup()

Telegram/SourceFiles/history/history_item.h

@@ -132,7 +132,10 @@ public:
 
 	// For edit media in history_message.
 	virtual void returnSavedMedia() {};
-	virtual void clearSavedMedia() {};
+	void clearSavedMedia() {
+		_isEditingMedia = false;
+		_savedMedia = nullptr;
+	}
 
 	// Zero result means this message is not self-destructing right now.
 	virtual crl::time getSelfDestructIn(crl::time now) {

Telegram/SourceFiles/history/history_message.cpp

@@ -955,14 +955,12 @@ void HistoryMessage::updateSentMedia(const MTPMessageMedia *media) {
 		}
 		_flags &= ~MTPDmessage_ClientFlag::f_from_inline_bot;
 	} else {
-		const auto shouldUpdate = _isEditingMedia ? true : !_media->updateSentMedia(*media);
 		if (_isEditingMedia) {
 			_savedMedia = _media->clone(this);
-		}
-		if (!media || !_media || shouldUpdate) {
+			refreshSentMedia(media);
+		} else if (!media || !_media || !_media->updateSentMedia(*media)) {
 			refreshSentMedia(media);
 		}
-		// _isEditingMedia = false;
 	}
 	history()->owner().requestItemResize(this);
 }

Telegram/SourceFiles/history/history_message.h

@@ -90,9 +90,6 @@ public:
 	void refreshMedia(const MTPMessageMedia *media);
 	void refreshSentMedia(const MTPMessageMedia *media);
 	void returnSavedMedia() override;
-	void clearSavedMedia() {
-		_savedMedia = nullptr;
-	}
 	void setMedia(const MTPMessageMedia &media);
 	static std::unique_ptr<Data::Media> CreateMedia(
 		not_null<HistoryMessage*> item,