Move start time for error timer into startBackground. (#3707)

mozilla-services · Nov 9, 2017 · 5a3e01c · 5a3e01c
1 parent 9a5a622
commit 5a3e01c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/addon/webextension/background/senderror.js b/addon/webextension/background/senderror.js
@@ -1,9 +1,7 @@
-/* globals analytics, communication, makeUuid, Raven, catcher, auth, log */
+/* globals startBackground, analytics, communication, makeUuid, Raven, catcher, auth, log */
 
 "use strict";
 
-const startTime = Date.now();
-
 this.senderror = (function() {
   let exports = {};
 
@@ -79,7 +77,7 @@ this.senderror = (function() {
         message = error.message;
       }
     }
-    if (Date.now() - startTime > 5 * 1000) {
+    if (Date.now() - startBackground.startTime > 5 * 1000) {
       browser.notifications.create(id, {
         type: "basic",
         // FIXME: need iconUrl for an image, see #2239

diff --git a/addon/webextension/background/startBackground.js b/addon/webextension/background/startBackground.js
@@ -6,9 +6,10 @@
    and loads the rest of the background page in response to those events, forwarding
    the events to main.onClicked, main.onClickedContextMenu, or communication.onMessage
 */
+const startTime = Date.now();
 
 this.startBackground = (function() {
-  let exports = {};
+  let exports = {startTime};
 
   const backgroundScripts = [
     "log.js",