Skip to content
  • Gijs Kruitbosch's avatar
    Bug 1618188 - remove XML backend for plugin and add-on blocklisting,... · f1a04bd3
    Gijs Kruitbosch authored
    Bug 1618188 - remove XML backend for plugin and add-on blocklisting, r=mconley,perftest-reviewers,whimboo
    
    This removes the obsolete backend. Notes on some of the less obvious changes
    made as part of this patch:
    
    - some of the gFoo style getters in Blocklist.jsm were only used by the XML
      version of the blocklist; I've removed them and tried to remove spurious
      settings of those properties in the remaining tests.
    - some utility methods (e.g. distribution information getters) were also only
      used for the XML version (for the update URL).
    - it's no longer necessary to test switching implementations.
    - in browser/base/content/test/plugins/, we ran some tests from two manifests
      in order to run them with both blocklist backends. The simplest way of
      reducing this back down to one was to remove the remote-settings one. If I'd
      been more future-oriented when I created the duplication, perhaps I would
      have moved the XML version out into a different manifest instead, but I
      didn't, so now it looks like we're removing the modern one, whereas really
      we're going to be running the modern one as part of the "normal" tests and
      we're no longer running the "old" tests.
    - removed all mentions I could see of extensions.blocklist.url which is no
      longer used for anything.
    - per https://bugzilla.mozilla.org/show_bug.cgi?id=1016555#c23, updated
      references for the OneCRL timing and how it relates to blocklist updates.
    
    Differential Revision: https://phabricator.services.mozilla.com/D64933
    
    --HG--
    extra : moz-landing-system : lando
    f1a04bd3