Issue 6599 - Hotfix to drop updated page everywhere

lib/subscriptionInit.js

Index: lib/subscriptionInit.js
===================================================================
--- a/lib/subscriptionInit.js
+++ b/lib/subscriptionInit.js
@@ -233,39 +233,19 @@
   for (let subscription of subscriptions)
   {
     FilterStorage.addSubscription(subscription);
     if (subscription instanceof DownloadableSubscription &&
         !subscription.lastDownload)
       Synchronizer.execute(subscription);
   }
 
-  if (!Prefs.suppress_first_run_page)
+  if (!Prefs.suppress_first_run_page && firstRun)
   {
-    let page = null;
-    if (firstRun)
-    {
-      page = "firstRun.html";
-    }
-    // For now we're limiting the updates page to users of
-    // Chromium-based browsers to gage its impact
-    else if (info.platform == "chromium" &&
-             updatesVersion > Prefs.last_updates_page_displayed)

[Sebastian Noack, 2018/04/23 19:17:06]

Nit: Please also remove the now unused import of updatesVersion from the top of
the file.

-    {
-      page = "updates.html";
-    }
-
-    if (page)
-    {
-      browser.tabs.create({url: browser.extension.getURL(page)});
-
-      // For new users and users that have already seen this updates page we
-      // want to avoid showing it again for subsequent updates.
-      Prefs.last_updates_page_displayed = updatesVersion;
-    }
+    browser.tabs.create({url: browser.extension.getURL("firstRun.html")});
   }
 
   initNotifications();
 }
 
 Promise.all([FilterNotifier.once("load"),
              Prefs.untilLoaded]).then(detectFirstRun)
                                 .then(getSubscriptions)