[rb] Add support for beta chrome

[aguspe]

User description

🔗 Related Issues

This PR introduces the changes that were originally supposed to be introduced in #15417 and reverted in #15837

💥 What does this PR do?

This PR adds beta Chrome support for the Ruby pipeline

🔧 Implementation Notes

The changes implemented introduced the download of the beta Chrome in pinned browsers and added it to the integration tests

💡 Additional Considerations

We started seeing memory heap issues, but that is due to Bazel and not this PR

---

PR Type

Enhancement

---

Description

• Add beta Chrome support to Ruby pipeline

  • Download and pin beta Chrome and Chromedriver
  • Integrate beta Chrome into Bazel build and test configs

• Update integration tests to run with beta Chrome

• Refactor and generalize Chrome pinning logic in scripts

• Update browser and driver versions for Edge and Firefox

---

Changes walkthrough 📝

	Relevant files
Enhancement	4 files browsers.bzl Add beta Chrome and Chromedriver data targets                        +22/-0    repositories.bzl Add beta Chrome/Chromedriver repos, update browser versions +91/-15  tests.bzl Add chrome-beta browser config for Ruby tests                        +25/-0    pinned_browsers.py Refactor and add beta Chrome/Chromedriver pinning logic    +42/-56
Configuration changes	1 files MODULE.bazel Register beta Chrome/Chromedriver in Bazel module                +4/-0
Tests	2 files BUILD.bazel Add chrome-beta to integration test browsers list                +1/-0      BUILD.bazel Run Chrome integration tests with chrome-beta                        +6/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Code Duplication The chrome() and chromedriver() functions contain duplicated code blocks with similar structure for handling Linux and Mac platforms. This could be refactored to use a common helper function to reduce duplication and improve maintainability. def chrome(selected_version, workspace_prefix=""): content = "" chrome_downloads = selected_version["downloads"]["chrome"] url = [d["url"] for d in chrome_downloads if d["platform"] == "linux64"][0] sha = calculate_hash(url) content += f""" http_archive( name = "linux_{workspace_prefix}chrome", url = "{url}", sha256 = "{sha}", build_file_content = \"\"\" load("@aspect_rules_js//js:defs.bzl", "js_library") package(default_visibility = ["//visibility:public"]) filegroup( name = "files", srcs = glob(["**/*"]), ) exports_files(["chrome-linux64/chrome"]) js_library( name = "chrome-js", data = [":files"], ) \"\"\", ) """ url = [d["url"] for d in chrome_downloads if d["platform"] == "mac-x64"][0] sha = calculate_hash(url) # Calculate SHA for Mac chrome content += f""" http_archive( name = "mac_{workspace_prefix}chrome", url = "{url}", sha256 = "{sha}", strip_prefix = "chrome-mac-x64", patch_cmds = [ "mv 'Google Chrome for Testing.app' Chrome.app", "mv 'Chrome.app/Contents/MacOS/Google Chrome for Testing' Chrome.app/Contents/MacOS/Chrome", ], build_file_content = \"\"\" load("@aspect_rules_js//js:defs.bzl", "js_library") package(default_visibility = ["//visibility:public"]) exports_files(["Chrome.app"]) js_library( name = "chrome-js", data = glob(["Chrome.app/**/*"]), ) \"\"\", ) """ return content Data Dependency The data dependency on "//common/extensions" was added for Chrome tests but it's not clear if this is required for both Chrome stable and beta. This dependency wasn't present in the original file and might need validation. data = ["//common/extensions"], )

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Define missing http variable The http variable is used but not defined in the scope. This will cause a NameError when the function is called. You need to initialize the http variable at the top of the file with the urllib3 library. scripts/pinned_browsers.py [28-32] def get_chrome_info_for_channel(channel): + import urllib3 + http = urllib3.PoolManager() r = http.request( "GET", f"https://chromiumdash.appspot.com/fetch_releases?channel={channel}&num=1&platform=Mac,Linux", ) Apply / Chat Suggestion importance[1-10]: 3 __ Why: While the http variable is used in the new function, the existing calculate_hash function also uses http without modification, suggesting it's likely defined elsewhere in the file outside the visible diff. The local import solution may be unnecessary if a global http variable already exists.	Low
Update