From e36aa524c9b473a8f2e7cbff229254f565e6eca9 Mon Sep 17 00:00:00 2001
From: Scott Nonnenberg <scott@nonnenberg.com>
Date: Tue, 25 Jul 2017 10:41:51 -0700
Subject: [PATCH] background.js: Flow promises properly in error case

FREEBIE
---
 js/background.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/js/background.js b/js/background.js
index 7212f5ffe..f75e6cce7 100644
--- a/js/background.js
+++ b/js/background.js
@@ -343,11 +343,15 @@
                     if (!conversation_timestamp || message_timestamp > conversation_timestamp) {
                         conversation.set({ timestamp: message.get('sent_at') });
                     }
-                    conversation.save();
+
                     conversation.trigger('newmessage', message);
                     if (initialLoadComplete) {
                         conversation.notify(message);
                     }
+
+                    return new Promise(function(resolve, reject) {
+                        conversation.save().then(resolve, reject);
+                    });
                 });
             });
         }