Backed out changeset 1256cadba71b (bug 1399796) for eslint failure at browser/modules/ContentCrashHandlers.jsm:675: Async method 'checkForUnsubmittedCrashReports' expected no return value. r=backout

browser/modules/ContentCrashHandlers.jsm

@@ -660,10 +660,6 @@ this.UnsubmittedCrashHandler = {
    *          If a notification cannot be shown, will resolve with null.
    */
   async checkForUnsubmittedCrashReports() {
-    if (!this.enabled || this.suppressed) {
-      return;
-    }
-
     let dateLimit = new Date();
     dateLimit.setDate(dateLimit.getDate() - PENDING_CRASH_REPORT_DAYS);
 

browser/modules/test/browser/browser_UnsubmittedCrashHandler.js

@@ -206,18 +206,6 @@ add_task(async function setup() {
   // disabled is an intentional choice, as this allows for easier
   // simulation of startup and shutdown.
   UnsubmittedCrashHandler.uninit();
-
-  // While we're here, let's test that we don't show the notification
-  // if we're disabled and something happens to check for unsubmitted
-  // crash reports.
-  await createPendingCrashReports(1);
-
-  notification =
-    await UnsubmittedCrashHandler.checkForUnsubmittedCrashReports();
-  Assert.ok(!notification, "There should not be a notification");
-
-  clearPendingCrashReports();
-
   await SpecialPowers.pushPrefEnv({
     set: [
       ["browser.crashReports.unsubmittedCheck.enabled", true],