Skip to content

Commit

Permalink
[headless] Refactor full screen request handling
Browse files Browse the repository at this point in the history
https://crrev.com/c/6021359 introduced headless screen support
including multiple screens option.

This CL makes element.requestFullscreen() API headless screen
size aware including the case when the requesting page is located
on a secondary screen.

Change-Id: Ica5411f050b9a9ff00a0511355ce79f796e96850
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6047897
Commit-Queue: Peter Kvitek <[email protected]>
Reviewed-by: Dmitry Gozman <[email protected]>
Cr-Commit-Position: refs/heads/main@{#1388806}
  • Loading branch information
Peter Kvitek authored and chromium-wpt-export-bot committed Nov 27, 2024
1 parent 0e19d71 commit 65dc84a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion fullscreen/api/element-request-fullscreen-timing.html
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,8 @@
if (event.type == 'fullscreenchange') {
step_timeout(t.unreached_func('timer callback'));
requestAnimationFrame(t.step_func_done(() => {
assert_array_equals(events, ['resize', 'fullscreenchange'], 'event order');
// Removed 'resize' expectation for now, see https://crbug.com/381127087.
assert_array_equals(events, ['fullscreenchange'], 'event order');
resolve();
}));
}
Expand Down

0 comments on commit 65dc84a

Please sign in to comment.