Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Addition: support geolocation emulation #183

Open
wants to merge 18 commits into
base: main
Choose a base branch
from

Conversation

sadym-chromium
Copy link

@sadym-chromium sadym-chromium commented Mar 17, 2025

EXPERIMENTAL! NOT READY FOR REVIEW YET!!!

Closes #146

The following tasks have been completed:

Implementation commitment (and no objections):

Documentation (new feature):

  • [ ] Updated implementation report N/A
  • [ ] Pinged MDN N/A. will be exposed via WebDriver BiDi
  • [ ] Added example to README or spec N/A. will be exposed via WebDriver BiDi

For documentation, either create an issue or pull request in MDN's Content repo - providing as much information as you can. PR is prefered.


Preview | Diff

@jgraham
Copy link
Member

jgraham commented Mar 18, 2025

At a high level this looks right to me (i.e. I think it matches the WebDriver API in which you provide either a full set of overrides, or null).

@reillyeon
Copy link
Member

As a general comment, please add the markup to indicate that these changes are "Candidate Additions". See the existing annotations in the spec for examples.

@sadym-chromium sadym-chromium changed the title [WIP] feat: support geolocation emulation Addition: support geolocation emulation Mar 19, 2025
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-emulation branch 3 times, most recently from e80cc2a to 8fc1684 Compare March 19, 2025 11:50
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-emulation branch from 8fc1684 to d0c2e91 Compare March 19, 2025 11:51
@sadym-chromium
Copy link
Author

As a general comment, please add the markup to indicate that these changes are "Candidate Additions". See the existing annotations in the spec for examples.

done

@sadym-chromium sadym-chromium marked this pull request as ready for review March 19, 2025 12:17
@sadym-chromium sadym-chromium marked this pull request as draft March 21, 2025 11:29
@sadym-chromium
Copy link
Author

Converting for draft to wait for the webdriver-bidi specification and corresponding WPT tests are landed.

@sadym-chromium sadym-chromium marked this pull request as ready for review March 24, 2025 13:15
@sadym-chromium
Copy link
Author

Webdriver-bidi is landed.

@reillyeon
Copy link
Member

In the "implementation commitment" section above, shouldn't we still make sure that browsers are committed to implementing the new WebDriver BiDi commands?

sadym-chromium and others added 8 commits March 28, 2025 22:00
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
Co-authored-by: Reilly Grant <[email protected]>
@reillyeon
Copy link
Member

This change looks good to me, however we still need to get implementation commitment signals from browser engines. Assuming no engine has already implemented the new WebDriver BiDi commands, please file issues for each engine and update the PR template with links to them.

Co-authored-by: Reilly Grant <[email protected]>
@sadym-chromium
Copy link
Author

This change looks good to me, however we still need to get implementation commitment signals from browser engines. Assuming no engine has already implemented the new WebDriver BiDi commands, please file issues for each engine and update the PR template with links to them.

done

@reillyeon
Copy link
Member

Why is WebKit marked "N/A"?

@sadym-chromium
Copy link
Author

Why is WebKit marked "N/A"?

I'm not aware of WebKit's commitment of supporting geolocation emulation, so I marked it as N/A. I guess the proper way to signal it is just to unmark the checkbox, which I just did.

@sadym-chromium sadym-chromium requested a review from reillyeon April 2, 2025 08:17
@reillyeon
Copy link
Member

Please file an issue against WebKit so we can get signals directly from the team. Or maybe @marcoscaceres can comment here.

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

Successfully merging this pull request may close these issues.

A means to test the API via Web Driver
4 participants