Issue 5316 - Adds dynamic filter types to devtools panel

messageResponder.js

Index: messageResponder.js
===================================================================
--- a/messageResponder.js
+++ b/messageResponder.js
@@ -37,16 +37,23 @@
 
   const info = require("info");
   const {
     Subscription,
     DownloadableSubscription,
     SpecialSubscription
   } = require("subscriptionClasses");
 
+  port.on("types.get", (message, sender) =>
+  {
+    let filterTypes = Array.from(require("requestBlocker").filterTypes);
+    filterTypes.push(filterTypes.splice(filterTypes.indexOf("OTHER"), 1)[0]);

[Sebastian Noack, 2017/09/11 16:44:14]

I would rather use the rest operator here. That way, in (the theoretical) case
there is no OTHER in filterTypes, you wouldn't add undefined to filterTypes.

  filterTypes.push(...filterTypes.splice(filterTypes.indexOf("OTHER"), 1));

[Jon Sonesen, 2017/09/12 11:42:41]

Oh, nifty. Thanks for pointing that out

+    return filterTypes;
+  });
+
   // Some modules doesn't exist on Firefox. Moreover,
   // require() throws an exception on Firefox in that case.
   // However, try/catch causes the whole function to to be
   // deoptimized on V8. So we wrap it into another function.
   function tryRequire(module)
   {
     try
     {