LGTM1 - I agree that this is small enough to just proceed.

/Daniel

On 2024-01-10 16:40, Noam Rosenthal wrote:


On Wed, Jan 10, 2024 at 3:32 PM Rick Byers <rby...@chromium.org> wrote:

    Hi Noam,
    This seems pretty trivial to me. The spec change is trivial and
    presumably any review feedback will only be editorial (not
    functional), so I'm OK not blocking approval on the spec PR
    landing. But could you get a WPTs implemented and at least ready
    to land (eg. along with the implementation CL) before we approve
    please?


Of course, already done:
https://github.com/web-platform-tests/wpt/pull/43919


    Thanks,
       Rick

    On Mon, Jan 8, 2024 at 12:10 PM Noam Rosenthal
    <nrosent...@chromium.org> wrote:


                Contact emails


                nrosent...@chromium.org


                Explainer


                None (this is a small change to an existing feature)


                Specification


                https://github.com/whatwg/html/pull/10035


                Summary


                Currently <script blocking="render"> requires a `src`
                attribute, even if this `src` is a data URI. This is
                an unnecessary constraint, as e.g. inline module
                scripts that import other script should still be able
                to render-block. See
                https://github.com/whatwg/html/issues/10034



                Blink component


                Blink
                
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink>


                TAG review


                None


                TAG review status


                Not applicable


                Risks




                Interoperability and Compatibility


                None



                /Gecko/: Positive
                
(https://github.com/whatwg/html/issues/10034#issuecomment-1881423778)
                Gecko folks confirmed that this can be included in
                their positive review for <script blocking=render>
                (which is not yet implemented)

                /WebKit/: No signal

                /Web developers/: No signals

                /Other signals/:


                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


                Is this feature fully tested by web-platform-tests
                
<https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>?


                No (will add as part of the implementation)


                Flag name on chrome://flags


                None


                Finch feature name


                None


                Non-finch justification


                None


                Requires code in //chrome?


                False


                Tracking bug


                https://bugs.chromium.org/p/chromium/issues/detail?id=1514881


                Estimated milestones


                Shipping on desktop     123

                Shipping on Android     123

                Shipping on WebView     123



                Anticipated spec changes


                See https://github.com/whatwg/html/pull/10035


                Link to entry on the Chrome Platform Status


                https://chromestatus.com/feature/5200457540829184

                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/CAJn%3DMYaABsXrGwCs5h14Vq5OJCStmPN-ki62-2NCQ6r79C9r9A%40mail.gmail.com
        
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJn%3DMYaABsXrGwCs5h14Vq5OJCStmPN-ki62-2NCQ6r79C9r9A%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/CAJn%3DMYZ8P1YZJ9Bhp4ZPo5RuRVkgkoCYdGi7K3kM58a7rSN%3Dcw%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAJn%3DMYZ8P1YZJ9Bhp4ZPo5RuRVkgkoCYdGi7K3kM58a7rSN%3Dcw%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/6a92a77f-81ce-4898-affe-ddbfdc2d30a6%40gmail.com.

Reply via email to