LGTM2

On 3/14/24 12:43 AM, Domenic Denicola wrote:
Awesome! LGTM1.

On Thu, Mar 14, 2024 at 1:35 PM 'Stephanie Zhang' via blink-dev <blink-dev@chromium.org> wrote:

    Thanks for clarifying! Updated the Chrome Status' "Finch Feature
    Name" field to kWritingSuggestions and removed the "Non-finch
    justification" field.

    On Wednesday, March 13, 2024 at 9:20:57 PM UTC-7 Domenic Denicola
    wrote:

        On Thu, Mar 14, 2024 at 12:54 PM 'Stephanie Zhang' via
        blink-dev <blin...@chromium.org> wrote:

            We do have a runtime feature flag 'WritingSuggestions
            
<https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5?q=writingsuggestions>'.
            We didn't think a Finch Trial was necessary, as the bulk
            of the changes were just adding the attribute and IDL
            functions
            <https://chromium-review.googlesource.com/c/chromium/src/+/5247315>.
            Since everything is implemented on the blink side, is a
            Finch feature flag still necessary? If it is, then I'll
            add that flag :)


        A runtime feature flag automatically generates
        
<https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/renderer/platform/runtime_enabled_features.json5;l=121-125;drc=36168a902bb7a33bfc8b46ad1f4ef6672872ad6d>
        a Finch flag, unless you turn that off :). So I think this is
        just a matter of updating the Chrome Status entry.


            On Wednesday, March 13, 2024 at 6:55:48 PM UTC-7 Domenic
            Denicola wrote:



                On Thursday, March 14, 2024 at 2:08:28 AM UTC+9
                Stephanie Zhang wrote:

                    *Contact emails*
                    
_sa...@microsoft.com_,_daniec@microsoft.com_,_stephanie.zh...@microsoft.com_

                    *Explainer*
                    
_https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/WritingSuggestions/explainer.md_
                    
<https://github.com/MicrosoftEdge/MSEdgeExplainers/blob/main/WritingSuggestions/explainer.md>

                    *Specification*
                    
_https://html.spec.whatwg.org/multipage/interaction.html#writing-suggestions_
                    <https://html.spec.whatwg.org/multipage/interaction.html>

                    *Summary*
                    UAs are starting to provide writing suggestions to
                    users as they type on various editable fields
                    across the web. While this is generally useful for
                    users, there are cases when developers may want to
                    turn off UA-provided writing assistance, such as
                    extensions or sites that wish to provide similar
                    functionality on their own. To that end,
                    developers need a solution that would turn on/off
                    UA-provided writing assistance. The new attribute
                    'writingsuggestions' has values 'true'/'false'
                    that would allow developers to turn on/off
                    browser-provided writing suggestions. The
                    attribute's state for an element can also be
                    inherited from ancestor elements, thereby allowing
                    developers to control this functionality at a
                    per-element or per-document/sub-document scale.


                    *Blink component*
                    _Blink>Editing_
                    
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EEditing>

                    *TAG review*
                    _https://github.com/w3ctag/design-reviews/issues/924_
                    <https://github.com/w3ctag/design-reviews/issues/924>

                    *TAG review status*
                    Issues addressed

                    *Risks*


                    *Interoperability and Compatibility*
                    None


                    /Gecko/: No signal
                    (_https://github.com/mozilla/standards-positions/issues/855_
                    <https://github.com/mozilla/standards-positions/issues/855>)

                    /WebKit/: In development
                    (_https://github.com/WebKit/standards-positions/issues/308_
                    <https://github.com/WebKit/standards-positions/issues/308>)
                    WebKit Implementation
                    PR:_https://github.com/WebKit/WebKit/pull/24051_
                    <https://github.com/WebKit/WebKit/pull/24051>

                    /Web developers/: No signals

                    /Other signals/:

                    *Ergonomics*
                    None


                    *Activation*
                    None


                    *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?/
                    None


                    *Debuggability*
                    None


                    *Will this feature be supported on all six Blink
                    platforms (Windows, Mac, Linux, ChromeOS, Android,
                    and Android WebView)?*
                    Yes
                    Attribute is available on all platforms.


                    *Is this feature fully tested
                    by****_web-platform-tests_*
                    
<https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>*?*
                    Yes
                    
_https://wpt.fyi/results/html/editing/editing-0/writing-suggestions/writingsuggestions.html?label=master&label=experimental&aligned_
                    
<https://wpt.fyi/results/html/editing/editing-0/writing-suggestions/writingsuggestions.html?label=master&label=experimental&aligned>


                    *Flag name on chrome://flags*
                    None

                    *Finch feature name*
                    None


                Per the flag guidelines
                
<https://chromium.googlesource.com/chromium/src/+/main/docs/flag_guarding_guidelines.md>,
                all new features are required to be placed behind a
                Finch feature flag (i.e. base::Feature flag). Can you
                ensure this is done and update the Chrome Status entry?


                    *Non-finch justification*
                    No finch trial needed.


                    *Requires code in //chrome?*
                    False

                    *Estimated milestones*
                    Shipping on desktop 124
                    Shipping on Android 124
                    Shipping on WebView 124


                    *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)./
                    None

                    *Link to entry on the Chrome Platform Status*
                    _https://chromestatus.com/feature/5153375153029120_
                    <https://chromestatus.com/feature/5153375153029120>

                    *Links to previous Intent discussions*
                    Intent to prototype:
                    
_https://groups.google.com/a/chromium.org/g/blink-dev/c/rHyRCx-hJhE_
                    
<https://groups.google.com/a/chromium.org/g/blink-dev/c/rHyRCx-hJhE>
                    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+...@chromium.org.
            To view this discussion on the web visit
            
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/8d6a0046-1a9b-44a4-9403-51067ca119fen%40chromium.org
            
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/8d6a0046-1a9b-44a4-9403-51067ca119fen%40chromium.org?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/adc944d3-853d-4964-9408-52b4014cc561n%40chromium.org
    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/adc944d3-853d-4964-9408-52b4014cc561n%40chromium.org?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/CAM0wra_CmAuu_4GVUtG%2BdNxLb8mB4oAXNXZ6PkGmP2jW-F0bxA%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAM0wra_CmAuu_4GVUtG%2BdNxLb8mB4oAXNXZ6PkGmP2jW-F0bxA%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/187f258c-c400-4721-b21b-38baac81161d%40chromium.org.

Reply via email to