Noissue - Add version check for runtime.openOptionsPage for Firefox for Android

ext/background.js

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-present eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 688 matching lines @@
 
   ext.showOptions = callback =>
   {
     let info = require("info");
 
     if ("openOptionsPage" in chrome.runtime &&
         // Some versions of Firefox for Android before version 57 do have a
         // runtime.openOptionsPage but it doesn't do anything.
         // https://bugzilla.mozilla.org/show_bug.cgi?id=1364945
         (info.application != "fennec" ||
-         (parseInt(info.applicationVersion, 10) >= 57 &&
-          info.applicationVersion != "57.0a1")))

[Wladimir Palant, 2017/08/25 09:56:46]

It should be sufficient to check that it's Firefox 57. That check will only be
wrong for users on older Firefox 57 nightly builds, which is usually ok because
nightly users update fairly quickly and there isn't a lot of them anyway.

[Manish Jethani, 2017/08/25 11:02:31]

Done.

+         parseInt(info.applicationVersion, 10) >= 57))
     {
       if (!callback)
       {
         chrome.runtime.openOptionsPage();
       }
       else
       {
         chrome.runtime.openOptionsPage(() =>
         {
           if (chrome.runtime.lastError)
@@ skipping 70 matching lines @@
   ext.windows = {
     create(createData, callback)
     {
       chrome.windows.create(createData, createdWindow =>
       {
         afterTabLoaded(callback)(createdWindow.tabs[0]);
       });
     }
   };
 }());