Skip to content

geolocation: Support errorCallback#42295

Merged
jdm merged 1 commit intoservo:mainfrom
arihant2math:geolocation-updates
Feb 3, 2026
Merged

geolocation: Support errorCallback#42295
jdm merged 1 commit intoservo:mainfrom
arihant2math:geolocation-updates

Conversation

@arihant2math
Copy link
Copy Markdown
Contributor

Support errorCallback in geolocation position request functions and throw the necessary errors.

Testing: Passes 3 more WPT tests
Fixes: Partially #38903

@servo-highfive servo-highfive added the S-awaiting-review There is new code that needs to be reviewed. label Feb 3, 2026
Copy link
Copy Markdown
Member

@jdm jdm left a comment

Choose a reason for hiding this comment

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

This is a reasonable workaround.

@servo-highfive servo-highfive removed the S-awaiting-review There is new code that needs to be reviewed. label Feb 3, 2026
@jdm jdm added this pull request to the merge queue Feb 3, 2026
@servo-highfive servo-highfive added the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 3, 2026
Merged via the queue into servo:main with commit 2c96178 Feb 3, 2026
29 checks passed
@servo-highfive servo-highfive removed the S-awaiting-merge The PR is in the process of compiling and running tests on the automated CI. label Feb 3, 2026
@arihant2math arihant2math deleted the geolocation-updates branch February 3, 2026 03:25
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.

3 participants