I've raised a potentially-breaking spec issue which I'd like to get a conclusion on before LGTMing this: https://github.com/WICG/ua-client-hints/issues/355
On Thursday, February 29, 2024 at 2:03:07 AM UTC+9 Yoav Weiss wrote: > LGTM2 > > On Wed, Feb 21, 2024 at 6:04 PM Vladimir Levin <vmp...@chromium.org> > wrote: > >> >> >> On Wed, Feb 21, 2024 at 11:25 AM Dustin Mitchell <djmit...@google.com> >> wrote: >> >>> Thanks for the comments! >>> >>> On Tue, Feb 20, 2024 at 11:10 AM Vladimir Levin <vmp...@chromium.org> >>> wrote: >>> >>>> >>>> >>>> On Tue, Feb 20, 2024 at 7:06 AM 'Dustin Mitchell' via blink-dev < >>>> blink-dev@chromium.org> wrote: >>>> >>>>> Contact emailsdjmit...@chromium.org >>>>> >>>>> Explainer >>>>> https://github.com/djmitche/web-explainers/blob/main/sec-ch-ua-form-factor.md >>>>> >>>>> Specification >>>>> https://wicg.github.io/ua-client-hints/#sec-ch-ua-form-factor >>>>> >>>>> Design docs >>>>> >>>>> https://github.com/djmitche/web-explainers/blob/main/sec-ch-ua-form-factor.md >>>>> >>>>> Summary >>>>> >>>>> This hint indicates the "form-factor" of the user-agent / device, so >>>>> that the site can tailor its response. >>>>> >>>>> >>>>> Blink componentBlink>Network >>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3ENetwork> >>>>> >>>>> Search tagsua-ch <https://chromestatus.com/features#tags:ua-ch>, uach >>>>> <https://chromestatus.com/features#tags:uach>, form-factor >>>>> <https://chromestatus.com/features#tags:form-factor> >>>>> >>>>> TAG reviewThis feature simply adds a new hint to the existing set of >>>>> hints, containing data that was previously represented in the user-agent >>>>> string. >>>>> >>>> >>>> Although this info may be available, I suspect this can be a new >>>> channel of information for clients that override the user-agent string or >>>> where this information isn't provided in the user-agent string. I don't >>>> believe this to be a problem, but just something to consider >>>> >>> >>> That's a good point, and likely needs to be considered for all client >>> hints. Do you think it's worth adding an issue in >>> https://github.com/WICG/ua-client-hints to track this? >>> >> >> I don't feel particularly strongly about this, but raising the issue to >> invite more opinions seems worthwhile. >> >> >>> >>>> >>>>> >>>>> >>>>> TAG review statusNot applicable >>>>> >>>>> Risks >>>>> >>>>> >>>>> Interoperability and Compatibility >>>>> >>>>> If other browsers do not implement this hint, then the information >>>>> will only be available in Chrome, and other browsers will implicitly >>>>> return >>>>> an empty value. >>>>> >>>>> >>>>> *Gecko*: No signal >>>>> >>>>> *WebKit*: No signal >>>>> >>>> >>>> Can you please file requests for positions for both Gecko and WebKit? >>>> >>> >>> Gecko: https://mozilla.github.io/standards-positions/#ua-client-hints >>> Webkit: >>> https://lists.webkit.org/pipermail/webkit-dev/2024-February/032618.html >>> >> >> Thank you for doing this. For posterity in the future, for WebKit, it >> might be better to file an issue here: >> https://github.com/WebKit/standards-positions/issues >> >> >>> >>> >>>> >>>> >>>>> *Web developers*: No signals >>>>> >>>>> *Other signals*: >>>>> >>>>> Ergonomics >>>>> >>>>> None - this fits with existing, similar client hints. >>>>> >>>>> >>>>> Activation >>>>> >>>>> No activation risks - developers will always need to handle the >>>>> situation where this hint is unavailable. >>>>> >>>>> >>>>> Security >>>>> >>>>> None >>>>> >>>>> >>>>> WebView application risks >>>>> >>>>> Does this intent deprecate or change behavior of existing APIs, such >>>>> that it has potentially high risk for Android WebView-based applications? >>>>> >>>>> No change to existing behavior; this adds a new hint which sites must >>>>> opt in to. Killswitch is available: ClientHintsFormFactor >>>>> >>>>> >>>>> Debuggability >>>>> >>>>> No changes in the checklist apply. >>>>> >>>>> >>>>> Will this feature be supported on all six Blink platforms (Windows, >>>>> Mac, Linux, ChromeOS, Android, and Android WebView)?Yes >>>>> >>>>> Is this feature fully tested by web-platform-tests >>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md> >>>>> ?Yes >>>>> >>>>> Included in existing tests for other client hints: >>>>> https://wpt.fyi/results/html/webappapis/system-state-and-capabilities/the-navigator-object/navigator_user_agent.https.html?label=experimental&label=master&aligned >>>>> >>>>> https://wpt.fyi/results/workers/WorkerNavigator_userAgentData.https.html?label=experimental&label=master&aligned >>>>> >>>>> >>>>> DevTrial instructions >>>>> https://github.com/djmitche/web-explainers/blob/main/sec-ch-ua-form-factor-howto.md >>>>> >>>>> Flag name on chrome://flagsNone >>>>> >>>>> Finch feature nameClientHintsFormFactor >>>>> >>>>> Requires code in //chrome?True >>>>> >>>>> Tracking bug >>>>> https://bugs.chromium.org/p/chromium/issues/detail?id=1442283 >>>>> >>>>> Launch bughttps://launch.corp.google.com/launch/4258684 >>>>> >>>>> MeasurementNone >>>>> >>>>> Availability expectationFeature is available only in Chromium >>>>> browsers for the foreseeable future. >>>>> >>>>> Adoption expectationFeature is used by specific partner(s) to provide >>>>> functionality within 12 months of launch in Chrome. >>>>> >>>>> Adoption planNew form-factors will add additional strings to this >>>>> hint, within the values given in the draft spec. The >>>>> ClientHintsXRFormFactor gates one such addition. >>>>> >>>>> Estimated milestones >>>>> Shipping on desktop 123 >>>>> DevTrial on desktop 122 >>>>> Shipping on Android 123 >>>>> DevTrial on Android 122 >>>>> Shipping on WebView 123 >>>>> >>>>> Anticipated spec changes >>>>> >>>>> Open questions about a feature may be a source of future web compat or >>>>> interop issues. Please list open issues (e.g. links to known github >>>>> issues >>>>> in the project for the feature specification) whose resolution may >>>>> introduce web compat/interop risk (e.g., changing to naming or structure >>>>> of >>>>> the API in a non-backward-compatible way). >>>>> No open issues >>>>> >>>>> Link to entry on the Chrome Platform Status >>>>> https://chromestatus.com/feature/5162545698045952 >>>>> >>>>> Links to previous Intent discussionsIntent to prototype: >>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CALMtyTTPUMfjo2v9s%3DKMALpS5MUcicvpM2ycNtz6GbRZ9JOaXQ%40mail.gmail.com >>>>> >>>>> This intent message was generated by Chrome Platform Status >>>>> <https://chromestatus.com/>. >>>>> >>>>> -- >>>>> You received this message because you are subscribed to the Google >>>>> Groups "blink-dev" group. >>>>> To unsubscribe from this group and stop receiving emails from it, send >>>>> an email to blink-dev+unsubscr...@chromium.org. >>>>> To view this discussion on the web visit >>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CALMtyTRPFmmHkrk24dSby%2B1bqTft%3DtUG-btxP_bg4Ckx4DNarw%40mail.gmail.com >>>>> >>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CALMtyTRPFmmHkrk24dSby%2B1bqTft%3DtUG-btxP_bg4Ckx4DNarw%40mail.gmail.com?utm_medium=email&utm_source=footer> >>>>> . >>>>> >>>> -- >> You received this message because you are subscribed to the Google Groups >> "blink-dev" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to blink-dev+unsubscr...@chromium.org. >> > To view this discussion on the web visit >> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CADsXd2M_0ZZJd9EOZ-8fwWgcnkSc9K1FufqVR4ZzwyMS7rJ4EA%40mail.gmail.com >> >> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CADsXd2M_0ZZJd9EOZ-8fwWgcnkSc9K1FufqVR4ZzwyMS7rJ4EA%40mail.gmail.com?utm_medium=email&utm_source=footer> >> . >> > -- You received this message because you are subscribed to the Google Groups "blink-dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to blink-dev+unsubscr...@chromium.org. To view this discussion on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/20c75133-f743-498b-be50-b24c27c7c999n%40chromium.org.