Bug 1237700 - JavaScript warning: resource://gre/modules/TelemetrySession.jsm, line 1783: unreachable code after return statement. r=gfritzsche
☠☠ backed out by 19a2342819e4 ☠ ☠
authorAlessio Placitelli <alessio.placitelli@gmail.com>
Fri, 08 Jan 2016 01:02:00 +0100
changeset 320133 f6447d37d113b86e2bf72356da1cb67dcd49ad97
parent 320132 4d4e15151ce32aa813a8a57e3ce56742db0f3a8f
child 320134 3e7dc6d87325ce8f6d817e7f0bad096152cc8aef
push id9143
push userahunt@mozilla.com
push dateFri, 08 Jan 2016 21:30:53 +0000
reviewersgfritzsche
bugs1237700
milestone46.0a1
Bug 1237700 - JavaScript warning: resource://gre/modules/TelemetrySession.jsm, line 1783: unreachable code after return statement. r=gfritzsche
toolkit/components/telemetry/TelemetrySession.jsm
--- a/toolkit/components/telemetry/TelemetrySession.jsm
+++ b/toolkit/components/telemetry/TelemetrySession.jsm
@@ -1774,19 +1774,16 @@ var Impl = {
         yield this.saveShutdownPings();
 
         if (IS_UNIFIED_TELEMETRY) {
           yield TelemetryController.removeAbortedSessionPing();
         }
 
         reset();
       }.bind(this));
-
-      reset();
-      return Promise.resolve();
     };
 
     // We can be in one the following states here:
     // 1) setupChromeProcess was never called
     // or it was called and
     //   2) _delayedInitTask was scheduled, but didn't run yet.
     //   3) _delayedInitTask is running now.
     //   4) _delayedInitTask finished running already.