Wait 5 seconds before showing any error notifications

This change eliminates the highly-visible class of bugs where an error notification is thrown at startup.
mozilla-services · Jun 9, 2017 · 9db01b7 · 9db01b7
1 parent 43977db
commit 9db01b7
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/addon/webextension/background/senderror.js b/addon/webextension/background/senderror.js
@@ -2,6 +2,8 @@
 
 "use strict";
 
+const startTime = Date.now();
+
 this.senderror = (function() {
   let exports = {};
 
@@ -73,12 +75,14 @@ this.senderror = (function() {
         message = error.message;
       }
     }
-    browser.notifications.create(id, {
-      type: "basic",
-      // FIXME: need iconUrl for an image, see #2239
-      title,
-      message
-    });
+    if (Date.now() - startTime > 5 * 1000) {
+      browser.notifications.create(id, {
+        type: "basic",
+        // FIXME: need iconUrl for an image, see #2239
+        title,
+        message
+      });
+    }
   };
 
   exports.reportError = function(e) {