[CRX] Fix feature detect of DNR responseHeaders option

Fix regression from mozilla#18711. `urlFilter` is a key of `condition`, not of the `rule`. Consequently, the feature detection method failed to detect the availability of the feature in Chrome 128+.
Rob--W · Sep 10, 2024 · bb04dd1 · bb04dd1
1 parent 9705169
commit bb04dd1
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/extensions/chromium/pdfHandler.js b/extensions/chromium/pdfHandler.js
@@ -227,8 +227,10 @@ async function isHeaderConditionSupported() {
       addRules: [
         {
           id: ruleId,
-          urlFilter: "|does_not_match_anything",
-          condition: { responseHeaders: [{ header: "whatever" }] },
+          condition: {
+            responseHeaders: [{ header: "whatever" }],
+            urlFilter: "|does_not_match_anything",
+          },
           action: { type: "block" },
         },
       ],
@@ -244,8 +246,10 @@ async function isHeaderConditionSupported() {
       addRules: [
         {
           id: ruleId,
-          urlFilter: "|does_not_match_anything",
-          condition: { responseHeaders: [] },
+          condition: {
+            responseHeaders: [],
+            urlFilter: "|does_not_match_anything",
+          },
           action: { type: "block" },
         },
       ],