Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-07 06:25:32 +01:00
Code Issues Projects Releases Packages Wiki Activity

Webdriver: limit tick action duration to ECMA maximum safe integer (#37977)

Browse source 
Workaround to prevent
`webdriver/tests/classic/perform_actions/invalid.py` from TIMEOUT. I
don't see the spec telling us to limit this number, so I assume the
correct fix is to handle it on parsing.

Testing: `webdriver/tests/classic/perform_actions/invalid.py`.

Signed-off-by: PotatoCP <kenzieradityatirtarahardja18@gmail.com>
This commit is contained in:
Kenzie Raditya Tirtarahardja 2025-07-11 11:23:49 +08:00 • committed by GitHub
parent 5b507dc871
commit 099d20fe94
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 10 additions and 55 deletions
Download patch file Download diff file Expand all files Collapse all files

10
components/webdriver_server/actions.rs
View file

@ -26,6 +26,9 @@ use crate::{Handler, VerifyBrowsingContextIsOpen, WebElement, wait_for_script_re
static POINTERMOVE_INTERVAL: u64 = 17;
static WHEELSCROLL_INTERVAL: u64 = 17;
// https://262.ecma-international.org/6.0/#sec-number.max_safe_integer
static MAXIMUM_SAFE_INTEGER: u64 = 9_007_199_254_740_991;
// A single action, corresponding to an `action object` in the spec.
// In the spec, `action item` refers to a plain JSON object.
// However, we use the name ActionItem here
@ -148,6 +151,13 @@ impl Handler {
// Step 1.2. Let tick duration be the result of
// computing the tick duration with argument tick actions.
let tick_duration = compute_tick_duration(tick_actions);
// FIXME: This is out of spec, but the test `perform_actions/invalid.py` requires
// that duration more than `MAXIMUM_SAFE_INTEGER` is considered invalid.
if tick_duration > MAXIMUM_SAFE_INTEGER {
return Err(ErrorStatus::InvalidArgument);
}
let now = Instant::now();
// Step 1.3. Try to dispatch tick actions