Skip to content

Conversation

navin772
Copy link
Member

@navin772 navin772 commented Aug 1, 2025

🔗 Related Issues

💥 What does this PR do?

Enable FedCM tests for chrome

🔧 Implementation Notes

💡 Additional Considerations

🔄 Types of changes

  • Cleanup (formatting, renaming)

@selenium-ci selenium-ci added the C-py Python Bindings label Aug 1, 2025
@navin772
Copy link
Member Author

navin772 commented Aug 1, 2025

As per https://issues.chromium.org/issues/425801332 this should be resolved once Chrome 140 is out. Manual test against chrome canary verifies this.

@cgoldberg
Copy link
Contributor

Nice. We might want to setup CI so we can test against beta releases of Chrome (I think Ruby bindings do this). I'm not sure if it would add too much noise... but it would be cool to see breaking changes and fixes in upcoming releases so we can be ready when they hit the stable channel.

@navin772
Copy link
Member Author

navin772 commented Aug 4, 2025

We already have http archives for linux_beta_chrome and its driver. I guess they were added when Ruby added chrome beta support.
I am working on adding chrome beta bazel targets for python in CI-RBE.

Currently, the bazel targets look like - //py:common-chrome-test/selenium/webdriver/common/fedcm_tests.py.
I will configure the beta ones to look like - //py:common-chrome-beta-test/selenium/webdriver/common/fedcm_tests.py.

I made some changes, I will open a draft PR to see if it works as intended.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-py Python Bindings
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants