Skip to content

feat: Introduce emulation.setClientHintsOverride#387

Merged
miketaylr merged 2 commits intoWICG:mainfrom
sadym-chromium:sadym/client-hints-emulation
Jan 16, 2026
Merged

feat: Introduce emulation.setClientHintsOverride#387
miketaylr merged 2 commits intoWICG:mainfrom
sadym-chromium:sadym/client-hints-emulation

Conversation

@sadym-chromium
Copy link
Contributor

@sadym-chromium sadym-chromium commented Jan 12, 2026

Addressing #386


Preview | Diff

…ding user agent client hints and refactor related internal references
@miketaylr miketaylr self-requested a review January 12, 2026 18:20
Copy link
Collaborator

@miketaylr miketaylr left a comment

Choose a reason for hiding this comment

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

Thank you! I'm less familiar with speccing the automation bits, but this LGTM.

@miketaylr miketaylr merged commit ea0eb00 into WICG:main Jan 16, 2026
2 checks passed
github-actions bot added a commit that referenced this pull request Jan 16, 2026
SHA: ea0eb00
Reason: push, by miketaylr

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
If the [=user agent=]'s [=emulated client hints=] is not null, the [=user agent=] MUST
return the corresponding value from the [=emulated client hints=] instead of the default
value for the requested property.
f
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@miketaylr oops! Fix is here: #388

@whimboo
Copy link

whimboo commented Jan 20, 2026

@sadym-chromium I was surprised to see that there is this PR which already got landed without discussing the implementation. Based on https://w3c.github.io/webdriver-bidi/#protocol-modules commands from external specifications should have their own module and should not extend existing modules. There is now issue #391 to fix that.

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