From 98403691d100fec0804c86d8b89d0d3f8da68354 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Ricardo=20Iv=C3=A1n=20Vieitez=20Parra?=
 <3857362+corrideat@users.noreply.github.com>
Date: Mon, 31 Jul 2023 23:22:07 +0200
Subject: fix: call setIsTrusted for generated events (MessageEvent) (#19919)

This addresses issue #19918.

## Issue description

Event messages have the wrong isTrusted value when they are not
triggered by user interaction, which differs from the browser. In
particular, all MessageEvents created by Deno have isTrusted set to
false, even though it should be true.

This is my first ever contribution to Deno, so I might be missing
something.
---
 ext/broadcast_channel/01_broadcast_channel.js | 2 ++
 ext/web/13_message_port.js                    | 3 +++
 ext/websocket/01_websocket.js                 | 3 +++
 3 files changed, 8 insertions(+)

(limited to 'ext')

diff --git a/ext/broadcast_channel/01_broadcast_channel.js b/ext/broadcast_channel/01_broadcast_channel.js
index d92aef072..ab315eaa6 100644
--- a/ext/broadcast_channel/01_broadcast_channel.js
+++ b/ext/broadcast_channel/01_broadcast_channel.js
@@ -8,6 +8,7 @@ import * as webidl from "ext:deno_webidl/00_webidl.js";
 import {
   defineEventHandler,
   EventTarget,
+  setIsTrusted,
   setTarget,
 } from "ext:deno_web/02_event.js";
 import DOMException from "ext:deno_web/01_dom_exception.js";
@@ -56,6 +57,7 @@ function dispatch(source, name, data) {
         data: core.deserialize(data), // TODO(bnoordhuis) Cache immutables.
         origin: "http://127.0.0.1",
       });
+      setIsTrusted(event, true);
       setTarget(event, channel);
       channel.dispatchEvent(event);
     };
diff --git a/ext/web/13_message_port.js b/ext/web/13_message_port.js
index 08f668b2d..40145db78 100644
--- a/ext/web/13_message_port.js
+++ b/ext/web/13_message_port.js
@@ -14,6 +14,7 @@ import {
   EventTarget,
   MessageEvent,
   setEventTargetData,
+  setIsTrusted,
 } from "ext:deno_web/02_event.js";
 import DOMException from "ext:deno_web/01_dom_exception.js";
 const primordials = globalThis.__bootstrap.primordials;
@@ -155,6 +156,7 @@ class MessagePort extends EventTarget {
           transferables = v[1];
         } catch (err) {
           const event = new MessageEvent("messageerror", { data: err });
+          setIsTrusted(event, true);
           this.dispatchEvent(event);
           return;
         }
@@ -165,6 +167,7 @@ class MessagePort extends EventTarget {
             (t) => ObjectPrototypeIsPrototypeOf(MessagePortPrototype, t),
           ),
         });
+        setIsTrusted(event, true);
         this.dispatchEvent(event);
       }
       this[_enabled] = false;
diff --git a/ext/websocket/01_websocket.js b/ext/websocket/01_websocket.js
index 3f4e16b90..71abb0830 100644
--- a/ext/websocket/01_websocket.js
+++ b/ext/websocket/01_websocket.js
@@ -17,6 +17,7 @@ import {
   Event,
   EventTarget,
   MessageEvent,
+  setIsTrusted,
 } from "ext:deno_web/02_event.js";
 import { Blob, BlobPrototype } from "ext:deno_web/09_file.js";
 import { getLocationHref } from "ext:deno_web/12_location.js";
@@ -430,6 +431,7 @@ class WebSocket extends EventTarget {
             data: op_ws_get_buffer_as_string(rid),
             origin: this[_url],
           });
+          setIsTrusted(event, true);
           dispatch(this, event);
           break;
         }
@@ -450,6 +452,7 @@ class WebSocket extends EventTarget {
             origin: this[_url],
             [_skipInternalInit]: true,
           });
+          setIsTrusted(event, true);
           dispatch(this, event);
           break;
         }
-- 
cgit v1.2.3