[ruby][BiDi] Browsing context commands

[TamsilAmani]

Description

Add BrowsingContext module commands based on the spec.
Refer to BiDi spec doc for details.

Motivation and Context

Support BrowsingContext commands based on the W3C BiDi spec. Currently, the commands supported by browsers are made public. Refer https://wpt.fyi/results/webdriver/tests/bidi/browsing_context?label=experimental&label=master&aligned&view=subtest. As browsers implement the rest of the commands, we can make them public (adding methods will not be a breaking change).

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[p0deje]

Great job!

May I suggest to re-organize the code a little bit? It would be nice to follow the naming structure where a folder corresponds to a namespace (class/module) and a file name corresponds to a class name. So in your PR it would be:

file path	full class name
rb/lib/selenium/webdriver/bidi/browsing_context/browsing_context.rb	Selenium::WebDriver::BiDi::BrowsingContext::BrowsingContext
rb/lib/selenium/webdriver/bidi/browsing_context/browsing_context_info.rb	Selenium::WebDriver::BiDi::BrowsingContext::BrowsingContextInfo
rb/lib/selenium/webdriver/bidi/browsing_context/navigate_result.rb	Selenium::WebDriver::BiDi::BrowsingContext::NavigateResult

It looks too verbose though, so I suggest the following structure:

file path	full class name
rb/lib/selenium/webdriver/bidi/browsing_context.rb	Selenium::WebDriver::BiDi::BrowsingContext
rb/lib/selenium/webdriver/bidi/browsing_context_info.rb	Selenium::WebDriver::BiDi::BrowsingContextInfo
rb/lib/selenium/webdriver/bidi/navigate_result.rb	Selenium::WebDriver::BiDi::NavigateResult

It would make the classes more aligned with the file paths they live in.

[TamsilAmani]

All changes have been incorporated. Thank you @p0deje :)

[pujagani]

Thank you @TamsilAmani!

[titusfortner]

@TamsilAmani this test is failing on all Chrome runs currently —

selenium/rb/spec/integration/selenium/webdriver/bidi_spec.rb

Line 36 in 36e0213

it 'can navigate and listen to errors' do