Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-10-01 00:59:15 +01:00
Code Issues Projects Releases Packages Wiki Activity

webdriver: Safely converting to OpenGL coordinates when taking (element) screenshot (#39543)

Browse source 
When taking element screenshot, we need to convert y-coordinates to
comply with OpenGL. With dpi > 1, sometimes y can be -1 due to rounding,
resulting in panic when
660f90f687/components/shared/compositing/rendering_context.rs (L655)

This PR safely converts.

Testing: When running in headed mode with dpi > 1,
`/webdriver/tests/classic/take_element_screenshot/scroll_into_view.py`
always passes locally now without panic.
Fixes the panic:
https://github.com/servo/servo/issues/39306#issuecomment-3342204869

Signed-off-by: Euclid Ye <yezhizhenjiakang@gmail.com>
This commit is contained in:
Euclid Ye 2025-09-28 12:50:16 +08:00 • committed by GitHub
parent 660f90f687
commit 41b1a8706b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
components/compositing/compositor.rs
View file

@ -1263,7 +1263,9 @@ impl IOCompositor {
let x = rect.origin.x as i32; let x = rect.origin.x as i32;
// We need to convert to the bottom-left origin coordinate // We need to convert to the bottom-left origin coordinate
// system used by OpenGL // system used by OpenGL
let y = (size.height as f32 - rect.origin.y - rect.size.height) as i32; // If dpi > 1, y can be computed to be -1 due to rounding issue, resulting in panic.
// https://github.com/servo/servo/issues/39306#issuecomment-3342204869
let y = 0.max((size.height as f32 - rect.origin.y - rect.size.height) as i32);
let w = rect.size.width as i32; let w = rect.size.width as i32;
let h = rect.size.height as i32; let h = rect.size.height as i32;