Technical review: Document ch-ua-high-entropy-values permissions-policy#42880
Conversation
chrisdavidmills
requested review from
hamishwillee
and removed request for
a team
github-actions
bot
added
Content:WebAPI
chrisdavidmills
changed the title
|
Preview URLs (5 pages)
External URLs (1)URL:
(comment last updated: 2026-01-26 22:23:15) |
files/en-us/web/api/navigatoruadata/gethighentropyvalues/index.md
Outdated
Show resolved
Hide resolved
files/en-us/web/api/navigatoruadata/gethighentropyvalues/index.md
Outdated
Show resolved
Hide resolved
| The default allowlist for `ch-ua-high-entropy-values` is `*`, which permits any content within the current document and all nested browsing contexts to use `getHighEntropyValues()`. | ||
|
|
||
| > [!NOTE] | ||
| > Access to individual high-entropy features can be controlled with their own [individual permissions policies](https://wicg.github.io/client-hints-infrastructure/#policy-controlled-features). |
There was a problem hiding this comment.
Yes, I still have to create something to catch these cases as discussed in #41648 (comment). Good enough for now.
files/en-us/web/http/reference/headers/permissions-policy/ch-ua-high-entropy-values/index.md
Show resolved
Hide resolved
…riginate Updated the description of the getHighEntropyValues() method to clarify its functionality and the types of values it returns.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
files/en-us/web/api/navigatoruadata/gethighentropyvalues/index.md
Outdated
Show resolved
Hide resolved
| {{APIRef("User-Agent Client Hints API")}}{{SeeCompatTable}}{{AvailableInWorkers}} | ||
|
|
||
| The **`getHighEntropyValues()`** method of the {{domxref("NavigatorUAData")}} interface is a {{jsxref("Promise")}} that resolves with a dictionary object containing the _high entropy_ values the user-agent returns. | ||
| The **`getHighEntropyValues()`** method of the {{domxref("NavigatorUAData")}} interface returns a {{jsxref("Promise")}} that resolves with a dictionary object containing information about the browser (user agent). |
There was a problem hiding this comment.
@chrisdavidmills I have rewritten the intro and minor parts below in 6e1bf42
Up front it wasn't clear that this returns the low entropy hints and some subset of the high entropy hints. That resulted in some confusion when you see that the parameters and return values don't match.
I have addressed that to my taste. Happy for you to modify as you wish, as long as that is clear ^^^. Either way, the changes I made now need review :-)
There was a problem hiding this comment.
Sure thing, thanks for the attention you've given this. The text was mostly fine, but I've made a few teaks. Most significantly, I updated the intro sentence to be more specific about what info it returns. In isolation, it was a bit vague; I'm thinking about people reading that intro sentence elsewhere, or just skimming the page.
There was a problem hiding this comment.
Let me know what you think, anyway.
There was a problem hiding this comment.
I think it is much better from your changes. I did a minor tweak/typo fix.
files/en-us/web/api/navigatoruadata/gethighentropyvalues/index.md
Outdated
Show resolved
Hide resolved
hamishwillee
left a comment
hamishwillee
left a comment
There was a problem hiding this comment.
Thanks @chrisdavidmills (and @miketaylr ). Looks good to me!
4 participants
Description
Chrome 144 adds support for the
ch-ua-high-entropy-valuesPermissions-Policydirective. See https://chromestatus.com/feature/6176703867781120.This PR adds a ref doc for the new directive, plus mentions in appropriate places.
@miketaylr, can I get your review on this one?
This PR also runs into this issue again, but I'm still not sure documenting those 22 individual policies is top priority. I can do it if you think it's important, but again, in a different PR.
Motivation
Additional details
Related issues and pull requests