[src/display/api.js] Simplify the sendTest function, used with Worker initialization (PR 14291 follow-up)
#14683
Conversation
…adow`ed getter work-arounds (PR 13813, 13882 follow-up) At the time private static class fields were to new, however that's no longer an issue and we can thus (ever so slightly) simplify the code.
…er initialization (PR 14291 follow-up) Given that we now only use Workers when `postMessage` transfers are supported, there's really no point in trying to send a "test" message *without* transfers present. Hence, if `postMessage` transfers are not supported by the browser, we'll now fallback to "fake" Workers immediately instead. The comment about Opera is also removed, since it was originally added back in PR 983 and mentions Opera `11.60` [which was released in 2011](https://en.wikipedia.org/wiki/History_of_the_Opera_web_browser#Version_11).
|
/botio test |
From: Bot.io (Windows)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.193.163.58:8877/067871d39cdc30b/output.txt |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @Snuffleupagus received. Current queue size: 0 Live output at: http://54.241.84.105:8877/96ff8d4d39ee98c/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/96ff8d4d39ee98c/output.txt Total script time: 0.01 mins |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/067871d39cdc30b/output.txt Total script time: 0.37 mins |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.241.84.105:8877/f8da0fdd536cf66/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/f8da0fdd536cf66/output.txt Total script time: 0.01 mins |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.241.84.105:8877/32f736e20e9bb9a/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/32f736e20e9bb9a/output.txt Total script time: 0.01 mins |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.241.84.105:8877/73381e2b1718c05/output.txt |
|
/botio test |
From: Bot.io (Linux m4)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.241.84.105:8877/3247220d31b2fd0/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @brendandahl received. Current queue size: 0 Live output at: http://54.193.163.58:8877/908d787e16884a1/output.txt |
From: Bot.io (Linux m4)FailedFull output at http://54.241.84.105:8877/3247220d31b2fd0/output.txt Total script time: 23.33 mins
Image differences available at: http://54.241.84.105:8877/3247220d31b2fd0/reftest-analyzer.html#web=eq.log |
From: Bot.io (Windows)FailedFull output at http://54.193.163.58:8877/908d787e16884a1/output.txt Total script time: 26.48 mins
|
|
Thank you! |
Given that we now only use Workers when
postMessagetransfers are supported, there's really no point in trying to send a "test" message without transfers present.Hence, if
postMessagetransfers are not supported by the browser, we'll now fallback to "fake" Workers immediately instead. The comment about Opera is also removed, since it was originally added back in PR #983 and mentions Opera11.60which was released in 2011.