From 39663e730ee3556afef0919da9b5822029a7687f Mon Sep 17 00:00:00 2001
From: Jonas Jenwald <jonas.jenwald@gmail.com>
Date: Wed, 4 Aug 2021 14:22:42 +0200
Subject: [PATCH] Change the `hashParameters` function to return a `Map` rather
 than an Object (issue 13862)

This patch (basically) mirrors the implementation in PR 13831, to get rid of the "Remote property injection" warning.
---
 test/resources/reftest-analyzer.js | 22 ++++++++++++----------
 1 file changed, 12 insertions(+), 10 deletions(-)

diff --git a/test/resources/reftest-analyzer.js b/test/resources/reftest-analyzer.js
index ffb54f66e..1587e05d7 100644
--- a/test/resources/reftest-analyzer.js
+++ b/test/resources/reftest-analyzer.js
@@ -52,24 +52,26 @@ window.onload = function () {
   }
 
   function hashParameters() {
-    const result = {};
-    const params = window.location.hash.substring(1).split(/[&;]/);
-    for (let i = 0; i < params.length; i++) {
-      const parts = params[i].split("=");
-      result[parts[0]] = unescape(unescape(parts[1]));
+    const query = window.location.hash.substring(1);
+    const params = new Map();
+    for (const part of query.split(/[&;]/)) {
+      const param = part.split("="),
+        key = param[0].toLowerCase(),
+        value = param.length > 1 ? param[1] : "";
+      params.set(decodeURIComponent(key), decodeURIComponent(value));
     }
-    return result;
+    return params;
   }
 
   function load() {
     gPhases = [ID("entry"), ID("loading"), ID("viewer")];
     buildMag();
     const params = hashParameters();
-    if (params.log) {
-      ID("logEntry").value = params.log;
+    if (params.has("log")) {
+      ID("logEntry").value = params.get("log");
       logPasted();
-    } else if (params.web) {
-      loadFromWeb(params.web);
+    } else if (params.has("web")) {
+      loadFromWeb(params.get("web"));
     }
     ID("logEntry").focus();
   }