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/CAOmohSKa9M%2BOko%3D8NM08KvTfGJU-NRVXQwan%3D4ND8WODk7FuXA%40mail.gmail.com.