Bug 843066 - [SMS] Callback in 'appendThreads' is not working properl…

…y r=julienw

apps/sms/js/sms.js

@@ -547,7 +547,8 @@ var ThreadListUI = {
     window.location.hash = '#thread-list';
   },
 
-  renderThreads: function thlui_renderThreads(threads, callback) {
+  renderThreads:
+    function thlui_renderThreads(threads, threadsRenderedCallback) {
     ThreadListUI.view.innerHTML = '';
 
     if (threads.length > 0) {
@@ -578,6 +579,10 @@ var ThreadListUI = {
       appendThreads(threads, function at_callback() {
         // Boot update of headers
         Utils.updateTimeHeaders();
+        // Once the rendering it's done, callback if needed
+        if (threadsRenderedCallback) {
+          threadsRenderedCallback();
+        }
       });
 
     } else {
@@ -590,11 +595,12 @@ var ThreadListUI = {
             '</div>';
       ThreadListUI.view.innerHTML = noResultHTML;
       ThreadListUI.iconEdit.classList.add('disabled');
-    }
-
-    // Callback when every thread is appended
-    if (callback) {
-      callback();
+      // Callback if exist
+      if (threadsRenderedCallback) {
+        setTimeout(function executeCB() {
+            threadsRenderedCallback();
+        });
+      }
     }
   },
 
@@ -1165,7 +1171,7 @@ var ThreadUI = {
     ThreadUI.scrollViewToBottom();
   },
   // Method for rendering the list of messages using infinite scroll
-  renderMessages: function thui_renderMessages(filter) {
+  renderMessages: function thui_renderMessages(filter, callback) {
     // We initialize all params before rendering
     this.initializeRendering();
     // We call getMessages with callbacks
@@ -1176,6 +1182,9 @@ var ThreadUI = {
       }
       // Update STATUS of messages if needed
       filter.read = false;
+      if (callback) {
+        callback();
+      }
       setTimeout(function updatingStatus() {
         var messagesUnreadIDs = [];
         var changeStatusOptions = {