Skip to content

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jan 19, 2025

Playwright on passing Headless: true uses the headless=old flag that is not supported in v>128. We can pass the flag on our own, both headless=new (only for v>128) and headless (all versions) work.

@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @akoeplinger, @matouskozak
See info in area-owners.md if you want to be subscribed.

@lewing
Copy link
Member

lewing commented Jan 21, 2025

 <launched> pid=951
        - [pid=951][err] Old Headless mode has been removed from the Chrome binary. Please use the new Headless mode (https://developer.chrome.com/docs/chromium/new-headless) or the chrome-headless-shell which is a standalone implementation of the old Headless mode (https://developer.chrome.com/blog/chrome-headless-shell).
        - [pid=951][err] 
      Stack Trace:
        /_/src/Playwright/Transport/Connection.cs(206,0): at Microsoft.Playwright.Transport.Connection.InnerSendMessageToServerAsync[T](ChannelOwner object, String method, Dictionary`2 dictionary, Boolean keepNulls)
        /_/src/Playwright/Transport/Connection.cs(532,0): at Microsoft.Playwright.Transport.Connection.WrapApiCallAsync[T](Func`1 action, Boolean isInternal)
        /_/src/Playwright/Core/BrowserType.cs(56,0): at Microsoft.Playwright.Core.BrowserType.LaunchAsync(BrowserTypeLaunchOptions options)
        /_/src/mono/wasm/Wasm.Build.Tests/BrowserRunner.cs(129,0): at Wasm.Build.Tests.BrowserRunner.SpawnBrowserAsync(String browserUrl, Boolean headless, Nullable`1 timeout, Int32 maxRetries, String locale)
        /_/src/mono/wasm/Wasm.Build.Tests/BrowserRunner.cs(165,0): at Wasm.Build.Tests.BrowserRunner.RunAsync(ToolCommand cmd, String args, Boolean headless, String locale, Action`2 onConsoleMessage, Action`1 onServerMessage, Action`1 onError, Func`2 modifyBrowserUrl)
        /_/src/mono/wasm/Wasm.Build.Tests/Templates/WasmTemplateTestsBase.cs(304,0): at Wasm.Build.Tests.WasmTemplateTestsBase.BrowserRunTest(String runArgs, String workingDirectory, RunOptions runOptions)
        /_/src/mono/wasm/Wasm.Build.Tests/Templates/WasmTemplateTestsBase.cs(329,0): at Wasm.Build.Tests.WasmTemplateTestsBase.BrowserRunTest(String runArgs, String workingDirectory, RunOptions runOptions)
        /_/src/mono/wasm/Wasm.Build.Tests/Templates/WasmTemplateTestsBase.cs(260,0): at Wasm.Build.Tests.WasmTemplateTestsBase.BrowserRun(RunOptions runOptions)
        /_/src/mono/wasm/Wasm.Build.Tests/Templates/WasmTemplateTestsBase.cs(255,0): at Wasm.Build.Tests.WasmTemplateTestsBase.RunForBuildWithDotnetRun(RunOptions runOptions)
        /_/src/mono/wasm/Wasm.Build.Tests/LazyLoadingTests.cs(42,0): at Wasm.Build.Tests.LazyLoadingTests.LoadLazyAssemblyBeforeItIsNeeded(String lazyLoadingTestExtension, String[] allLazyLoadingTestExtensions)

@ilonatommy ilonatommy self-assigned this Jan 22, 2025
@ilonatommy
Copy link
Member

ilonatommy commented Jan 22, 2025

Playwright does not support the new headless mode out of the box if we are using a custom path to the browser doc.

The best fix is to update the urls that we are pulling chrome from to match the new chrome-headless-shell releases.

Edit: for chromium that we're using in tests, shell version is not shipped. Instead, we should use headless=new or headless.

@ilonatommy
Copy link
Member

Performance failures are not related. Checked on a playground PR: #111747.

@ilonatommy
Copy link
Member

/ba-g failures are not related

@ilonatommy ilonatommy merged commit 85506c3 into main Jan 24, 2025
32 of 38 checks passed
@jkotas jkotas deleted the update-chrome-version-12848518924 branch February 2, 2025 18:48
@github-actions github-actions bot locked and limited conversation to collaborators Mar 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants