mirrors/ladybird
0
0
Fork 0
mirror of https://github.com/LadybirdBrowser/ladybird.git synced 2025-07-29 04:09:13 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 9

Tests: Rearrange message order in Messaging-post-channel-over-channel

Browse source 
The way this test was written didn't guarantee a deterministic message
order, resulting in different output in Chromium and Firefox. This
change slightly rearranges the message order to make it deterministic.

This change is necessary as a prepartion for upcoming change that makes
MessagePort post messages from a separate thread, which would've
revealed the non-deterministic message order.
This commit is contained in:
Aliaksandr Kalenik 2025-04-08 16:41:45 +02:00 • committed by Alexander Kalenik
commit af2dae63d1
Notes: github-actions[bot] 2025-04-08 19:10:37 +00:00
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

2
Tests/LibWeb/Text/expected/Messaging/Messaging-post-channel-over-channel.txt
View file

@ -1,6 +1,6 @@
Port1: "Hello" Port1: "Hello"
Port1: {"foo":{}} Port1: {"foo":{}}
Port1: "DONE"
Port2: "Hello" Port2: "Hello"
Port3: "Hello from the transferred port" Port3: "Hello from the transferred port"
Port1: "DONE"
Port2: "DONE" Port2: "DONE"

4
Tests/LibWeb/Text/input/Messaging/Messaging-post-channel-over-channel.html
View file

@ -23,11 +23,11 @@
let channel2 = new MessageChannel(); let channel2 = new MessageChannel();
channel2.port2.onmessage = (event) => { channel2.port2.onmessage = (event) => {
println("Port3: " + JSON.stringify(event.data)) println("Port3: " + JSON.stringify(event.data));
channel.port2.postMessage("DONE");
} }
channel.port2.postMessage("Hello"); channel.port2.postMessage("Hello");
channel.port2.postMessage({ foo: channel2.port1 }, { transfer: [channel2.port1] }); channel.port2.postMessage({ foo: channel2.port1 }, { transfer: [channel2.port1] });
channel.port2.postMessage("DONE");
}); });
</script> </script>