From 746b1b257982693aa487fe37a608bf794e560935 Mon Sep 17 00:00:00 2001 From: Charles Kerr Date: Thu, 23 Apr 2026 17:56:37 -0500 Subject: [PATCH] chore: temporarily disable new flaky test (#51295) --- spec/api-web-contents-view-spec.ts | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/spec/api-web-contents-view-spec.ts b/spec/api-web-contents-view-spec.ts index f5a85271ff..40db7686bc 100644 --- a/spec/api-web-contents-view-spec.ts +++ b/spec/api-web-contents-view-spec.ts @@ -256,12 +256,16 @@ describe('WebContentsView', () => { await p; expect(await v.webContents.executeJavaScript('document.visibilityState')).to.equal('visible'); - const viewportSize = await v.webContents.executeJavaScript( - '({ width: window.innerWidth, height: window.innerHeight })' - ); - const contentBounds = w.getContentBounds(); - expect(viewportSize.width).to.equal(contentBounds.width); - expect(viewportSize.height).to.equal(contentBounds.height); + // These two new `expect` calls added in 2026-04-20 @ 2e17a57 are causing + // CI flakes. https://github.com/electron/electron/issues/51228 + // TODO(ckerr) fix the flakes and re-enable + // + // const viewportSize = await v.webContents.executeJavaScript( + // '({ width: window.innerWidth, height: window.innerHeight })' + // ); + // const contentBounds = w.getContentBounds(); + // expect(viewportSize.width).to.equal(contentBounds.width); + // expect(viewportSize.height).to.equal(contentBounds.height); }); it('is initially visible if load happens after attach', async () => {