Skip to content

Comments

emulate geolocation POSITION_UNAVAILABLE error#911

Merged
sadym-chromium merged 7 commits intomainfrom
sadym/geolocation-error
Apr 30, 2025
Merged

emulate geolocation POSITION_UNAVAILABLE error#911
sadym-chromium merged 7 commits intomainfrom
sadym/geolocation-error

Conversation

@sadym-chromium
Copy link
Contributor

@sadym-chromium sadym-chromium commented Apr 29, 2025

WPT tests: web-platform-tests/wpt#52250

I don't think we need to emulate other errors (PERMISSION_DENIED, TIMEOUT), as it can be tested without emulation.

The geolocation emulation spec was written to support any kind of error emulations, which is an overkill, and is tricky to be implemented, so I propose to limit the error emulation only to "POSITION_UNAVAILABLE" on the WebDriver BiDi side.

From geolocation spec:

If emulatedPositionData is a GeolocationPositionError:
Call back with error passing errorCallback and emulatedPositionData.

nd fix `altitudeAccuracy` + `altitude` access path
@sadym-chromium sadym-chromium requested review from OrKoN and lutien April 29, 2025 11:44
@sadym-chromium sadym-chromium changed the title emulate geolocation error emulate geolocation POSITION_UNAVAILABLE error Apr 29, 2025
@sadym-chromium sadym-chromium requested a review from OrKoN April 29, 2025 17:26
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-error branch 3 times, most recently from 5d2968a to e8f8d62 Compare April 29, 2025 17:52
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-error branch from e8f8d62 to 2abb684 Compare April 29, 2025 17:53
Co-authored-by: Henrik Skupin <mail@hskupin.info>
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-error branch from 1ce3afb to c91ae18 Compare April 30, 2025 09:54
@sadym-chromium sadym-chromium force-pushed the sadym/geolocation-error branch from 1c8848b to c6a9a9e Compare April 30, 2025 10:11
@sadym-chromium sadym-chromium requested a review from whimboo April 30, 2025 10:21
@sadym-chromium
Copy link
Contributor Author

@juliandescottes any other concerns?

Copy link
Contributor

@juliandescottes juliandescottes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@sadym-chromium sadym-chromium merged commit 5e3b177 into main Apr 30, 2025
5 checks passed
@sadym-chromium sadym-chromium deleted the sadym/geolocation-error branch April 30, 2025 13:55
github-actions bot added a commit that referenced this pull request Apr 30, 2025
SHA: 5e3b177
Reason: push, by sadym-chromium

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
sadym-chromium added a commit to GoogleChromeLabs/chromium-bidi that referenced this pull request Apr 30, 2025
sadym-chromium added a commit to web-platform-tests/wpt that referenced this pull request May 5, 2025
lando-prod-mozilla bot pushed a commit to mozilla-firefox/firefox that referenced this pull request May 12, 2025
…ilable",

Automatic update from web-platform-tests
[wdspec] test geolocation "positionUnavailable" (#52250)

Spec: w3c/webdriver-bidi#911
--

wpt-commits: 4e5ff79b8032c5762b1548341324a7d357301c2d
wpt-pr: 52250

Differential Revision: https://phabricator.services.mozilla.com/D248839
moz-v2v-gh pushed a commit to mozilla/gecko-dev that referenced this pull request May 12, 2025
…ilable",

Automatic update from web-platform-tests
[wdspec] test geolocation "positionUnavailable" (#52250)

Spec: w3c/webdriver-bidi#911
--

wpt-commits: 4e5ff79b8032c5762b1548341324a7d357301c2d
wpt-pr: 52250

Differential Revision: https://phabricator.services.mozilla.com/D248839
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified-and-comments-removed that referenced this pull request May 14, 2025
…ilable",

Automatic update from web-platform-tests
[wdspec] test geolocation "positionUnavailable" (#52250)

Spec: w3c/webdriver-bidi#911
--

wpt-commits: 4e5ff79b8032c5762b1548341324a7d357301c2d
wpt-pr: 52250

Differential Revision: https://phabricator.services.mozilla.com/D248839

UltraBlame original commit: 3870830f0eaf15001a0f63b3dc530a0616462516
gecko-dev-updater pushed a commit to marco-c/gecko-dev-comments-removed that referenced this pull request May 14, 2025
…ilable",

Automatic update from web-platform-tests
[wdspec] test geolocation "positionUnavailable" (#52250)

Spec: w3c/webdriver-bidi#911
--

wpt-commits: 4e5ff79b8032c5762b1548341324a7d357301c2d
wpt-pr: 52250

Differential Revision: https://phabricator.services.mozilla.com/D248839

UltraBlame original commit: 3870830f0eaf15001a0f63b3dc530a0616462516
gecko-dev-updater pushed a commit to marco-c/gecko-dev-wordified that referenced this pull request May 14, 2025
…ilable",

Automatic update from web-platform-tests
[wdspec] test geolocation "positionUnavailable" (#52250)

Spec: w3c/webdriver-bidi#911
--

wpt-commits: 4e5ff79b8032c5762b1548341324a7d357301c2d
wpt-pr: 52250

Differential Revision: https://phabricator.services.mozilla.com/D248839

UltraBlame original commit: 3870830f0eaf15001a0f63b3dc530a0616462516
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.

4 participants