Issue 4722 - Drop support for Chrome 41 - 48

chrome/ext/background.js

Index: chrome/ext/background.js
diff --git a/chrome/ext/background.js b/chrome/ext/background.js
index 49c0366bfcfe9de61f4b472c0131ea9f3ba614e8..4681ec08bb0fbf200b337c74cd8d33e09d754c26 100644
--- a/chrome/ext/background.js
+++ b/chrome/ext/background.js
@@ -202,38 +202,11 @@
     this._changes = null;
   };
   BrowserAction.prototype = {
-    _legacySetIcon: function(details)
-    {
-      var legacyDetails = {};
-      for (var key in details)
-      {
-        var value = details[key];
-        if (typeof value == "object")
-          value = {19: value[19], 38: value[38]};
-        legacyDetails[key] = value;
-      }
-      chrome.browserAction.setIcon(legacyDetails);
-    },
-    _safeSetIcon: function(details)
-    {
-      try
-      {
-        chrome.browserAction.setIcon(details);
-      }
-      catch (e)
-      {
-        // Older versions of Chrome do not allow any sizes other than 19 and 38
-        // to be present, but newer versions of Chrome (and Edge) prefer
-        // different sizes.
-        this._safeSetIcon = this._legacySetIcon;
-        this._legacySetIcon(details);
-      }
-    },
     _applyChanges: function()
     {
       if ("iconPath" in this._changes)
       {
-        this._safeSetIcon({
+        chrome.browserAction.setIcon({
           tabId: this._tabId,
           path: {
             16: this._changes.iconPath.replace("$size", "16"),
@@ -438,33 +411,6 @@
       var onBeforeNavigate = chrome.webNavigation.onBeforeNavigate;
       if (!onBeforeNavigate.hasListener(propagateHandlerBehaviorChange))
         onBeforeNavigate.addListener(propagateHandlerBehaviorChange);
-    },
-    getIndistinguishableTypes: function()
-    {
-      // Chrome 38-48 mistakenly reports requests of type `object`
-      // (e.g. requests initiated by Flash) with the type `other`.
-      // https://code.google.com/p/chromium/issues/detail?id=410382
-      var match = navigator.userAgent.match(/\bChrome\/(\d+)/);
-      if (match)
-      {
-        var version = parseInt(match[1], 10);
-        if (version >= 38 && version <= 48)
-          return [["OTHER", "OBJECT", "OBJECT_SUBREQUEST"]];
-      }
-
-      // Chrome <44 doesn't have ResourceType.
-      var ResourceType = chrome.webRequest.ResourceType || {};
-
-      // Before Chrome 49, requests of the type `font` and `ping`
-      // have been reported with the type `other`.
-      // https://code.google.com/p/chromium/issues/detail?id=410382
-      var otherTypes = ["OTHER", "MEDIA"];
-      if (!("FONT" in ResourceType))
-        otherTypes.push("FONT");
-      if (!("PING" in ResourceType))
-        otherTypes.push("PING");
-
-      return [["OBJECT", "OBJECT_SUBREQUEST"], otherTypes];

[Wladimir Palant, 2016/12/13 14:15:02]

OBJECT and OBJECT_SUBREQUEST are still indistinguishable as of Chrome 55 and
this is unlikely to change. Looks like you removed a bit too much code.

[Sebastian Noack, 2016/12/13 14:35:39]

Yep. But let's hard-code that like we initially did, and still get rid of
getIndistinguishableTypes() which mainly existed to abstract the differences
between Chrome and Safari here.

[Wladimir Palant, 2016/12/13 14:46:35]

Not sure whether that's a good idea - with Firefox and Edge we might get more
complicated logic here again soon.

[kzar, 2016/12/13 15:05:40]

Done.

[Sebastian Noack, 2016/12/13 15:06:58]

Well, ext is matter to go away, not that we need any platform-level abstraction
for that. That mapping could/should be performed at runtime (outside of ext)
based on the values you'll find in webRequest.ResourceType:
https://developer.mozilla.org/en-US/Add-ons/WebExtensions/API/webRequest/Reso...

[kzar, 2016/12/13 15:12:54]

I don't mind too much either way, if it's hard-coded or not, but perhaps
Wladimir has a point since we're likely adding the Edge and Firefox platforms in
the near future. Do you insist I remove the function?

[Sebastian Noack, 2016/12/14 12:39:53]

Again, the differences in Firefox and Edge aren't meant to be abstracted by yet
another ext backend, as we agreed to get rid of ext altogether eventually. In
the particular case here there is no need for platform specific code anyway.
Yes, Firefox uses a different (more granular) set of request types. But there is
no need for a special-case for Firefox. Instead, you can and should do feature
detection and map the values given in webRequest.ResourceType accordingly. This
can be done in lib/requestBlocker.js.

[kzar, 2016/12/16 09:38:35]

Done.

     }
   };
 
@@ -554,22 +500,9 @@
           if (!frames)
             return null;
 
-          if ("frameId" in rawSender)
-          {
-            // Chrome 41+
-            var frame = frames[rawSender.frameId];
-            if (frame)
-              return frame.parent;
-          }
-          else
-          {
-            // Chrome 28-40
-            for (var frameId in frames)
-            {
-              if (frames[frameId].url.href == this.url.href)
-                return frames[frameId].parent;
-            }
-          }
+          var frame = frames[rawSender.frameId];
+          if (frame)
+            return frame.parent;
 
           return frames[0];
         }