mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

A number of performance tweaks

Browse source
This commit is contained in:
Scott Nonnenberg 2019-09-06 12:16:54 -07:00
parent 07689379cb
commit e011589a5e
3 changed files with 29 additions and 28 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
js/models/conversations.js
View file

@ -409,12 +409,14 @@
await this.initialPromise; await this.initialPromise;
const verified = await this.safeGetVerified(); const verified = await this.safeGetVerified();
this.set({ verified }); if (this.get('verified') !== verified) {
this.set({ verified });
// we don't await here because we don't need to wait for this to finish // we don't await here because we don't need to wait for this to finish
window.Signal.Data.updateConversation(this.id, this.attributes, { window.Signal.Data.updateConversation(this.id, this.attributes, {
Conversation: Whisper.Conversation, Conversation: Whisper.Conversation,
}); });
}
return; return;
} }

2
js/models/messages.js
View file

@ -516,7 +516,7 @@
findAndFormatContact(phoneNumber) { findAndFormatContact(phoneNumber) {
const contactModel = this.findContact(phoneNumber); const contactModel = this.findContact(phoneNumber);
if (contactModel) { if (contactModel) {
return contactModel.getProps(); return contactModel.format();
} }
const { format } = PhoneNumber; const { format } = PhoneNumber;

43
js/views/conversation_view.js
View file

@ -178,10 +178,12 @@
this.model.updateVerified.bind(this.model), this.model.updateVerified.bind(this.model),
1000 // one second 1000 // one second
); );
this.throttledGetProfiles = _.throttle( this.model.throttledGetProfiles =
this.model.getProfiles.bind(this.model), this.model.throttledGetProfiles ||
1000 * 60 * 5 // five minutes _.throttle(
); this.model.getProfiles.bind(this.model),
1000 * 60 * 5 // five minutes
);
this.debouncedMaybeGrabLinkPreview = _.debounce( this.debouncedMaybeGrabLinkPreview = _.debounce(
this.maybeGrabLinkPreview.bind(this), this.maybeGrabLinkPreview.bind(this),
200 200
@ -1500,23 +1502,6 @@
}, },
async onOpened(messageId) { async onOpened(messageId) {
this.openStart = Date.now();
this.lastActivity = Date.now();
this.focusMessageField();
this.model.updateLastMessage();
const statusPromise = this.throttledGetProfiles();
// eslint-disable-next-line more/no-then
this.statusFetch = statusPromise.then(() =>
// eslint-disable-next-line more/no-then
this.model.updateVerified().then(() => {
this.onVerifiedChange();
this.statusFetch = null;
window.log.info('done with status fetch');
})
);
if (messageId) { if (messageId) {
const message = await getMessageById(messageId, { const message = await getMessageById(messageId, {
Message: Whisper.Message, Message: Whisper.Message,
@ -1532,10 +1517,24 @@
this.loadNewestMessages(); this.loadNewestMessages();
this.focusMessageField();
const quotedMessageId = this.model.get('quotedMessageId'); const quotedMessageId = this.model.get('quotedMessageId');
if (quotedMessageId) { if (quotedMessageId) {
this.setQuoteMessage(quotedMessageId); this.setQuoteMessage(quotedMessageId);
} }
this.model.updateLastMessage();
const statusPromise = this.model.throttledGetProfiles();
// eslint-disable-next-line more/no-then
this.statusFetch = statusPromise.then(() =>
// eslint-disable-next-line more/no-then
this.model.updateVerified().then(() => {
this.onVerifiedChange();
this.statusFetch = null;
})
);
}, },
async retrySend(messageId) { async retrySend(messageId) {
@ -2424,7 +2423,7 @@
const trimmed = const trimmed =
messageText && messageText.length > 0 ? messageText.trim() : ''; messageText && messageText.length > 0 ? messageText.trim() : '';
if ((this.model.get('draft') && !messageText) || trimmed.length === 0) { if (this.model.get('draft') && (!messageText || trimmed.length === 0)) {
this.model.set({ this.model.set({
draft: null, draft: null,
}); });