New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: html fullscreen transitions stacking #34909
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are some related macOS failures
const width = await w.webContents.executeJavaScript( | ||
"document.querySelector('iframe').offsetWidth" | ||
); | ||
expect(width).to.equal(0); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This assertion is failing on macOS builds:
not ok 1114 iframe using HTML fullscreen API while window is OS-fullscreened can fullscreen from out-of-process iframes (macOS)
AssertionError: expected 1024 to equal 0
at Context.<anonymous> (electron/spec-main/chromium-spec.ts:1701:22)
// TODO(jkleinsc) fix this flaky test on WOA | ||
ifit(process.platform !== 'win32' || process.arch !== 'arm64')('can fullscreen from in-process iframes', async () => { | ||
if (process.platform === 'darwin') await emittedOnce(w, 'enter-full-screen'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this event is not emitted and so, this test times out on macOS:
not ok 1115 iframe using HTML fullscreen API while window is OS-fullscreened can fullscreen from in-process iframes
Error: Timeout of 30000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/Users/distiller/project/src/electron/spec-main/chromium-spec.ts)
at listOnTimeout (node:internal/timers:557:17)
at processTimers (node:internal/timers:500:7)
Gonna leave this in 19 and 20. Forgot this hinged on an upstream change not in 18. |
Backport of #32905.
See that PR for details.
Notes: Fixes an issue with fullscreen transitions when HTML fullscreen is requested.