LGTM3

/Daniel

On 2023-06-07 17:37, Philip Jägenstedt wrote:
LGTM2

On Wed, Jun 7, 2023 at 5:36 PM Chris Harrelson <chris...@chromium.org> wrote:

    Thanks! LGTM1

    On Wed, Jun 7, 2023 at 6:55 AM Ari Chivukula
    <aric...@chromium.org> wrote:

        I'm back! The code has been merged behind a flag w/ wpts:
        https://chromium-review.googlesource.com/c/chromium/src/+/4538244

        The spec changes have been merged too:
        https://github.com/w3c/navigation-timing/pull/188
        https://github.com/WICG/client-hints-infrastructure/pull/153

        ~ Ari Chivukula (Their/There/They're)


        On Wed, Apr 19, 2023 at 12:02 PM Rick Byers
        <rby...@chromium.org> wrote:

            Great, thanks Ari! No worries at all, checking to make
            sure the i's have been dotted and t's have been crossed is
            what the API owners are here for :-)

            On Wed, Apr 19, 2023 at 11:59 AM Ari Chivukula
            <aric...@chromium.org> wrote:

                Works for me, didn't realize that was the standard order.

                ~ Ari Chivukula (Their/There/They're)


                On Wed, Apr 19, 2023 at 11:58 AM Rick Byers
                <rby...@chromium.org> wrote:

                    Hi Ari,
                    The documented process
                    
<https://www.chromium.org/blink/launching-features/#new-feature-prepare-to-ship>
 is
                    to land features and specs before requesting
                    approval to ship, as that gives the best possible
                    chance for wide review and eventual interop. But
                    we unfortunately often find ourselves in the
                    position that this is impractical - eg. spec work
                    blocked from landing due to lack of interest from
                    a 2nd implementor (WHATWG venue) or just very slow
                    reviews. So, if necessary, we are willing to
                    approve based on a PR only but it's usually
                    considered an exceptional situation. None of those
                    seem to be the case here, it's just blocked on
                    some impl and spec work right? In particular, we
                    definitely want to see WPTs landed prior to
                    shipping, so we're aligned with Noam on that
                    request. chrishtr, foolip, yoav and I just
                    discussed this now in the API owners meeting and
                    are aligned on this being standard practice
                    (though wouldn't be surprised if we've slipped up
                    occasionally).

                    Thanks,
                       Rick

                    On Wed, Apr 19, 2023 at 11:34 AM Ari Chivukula
                    <aric...@chromium.org> wrote:

                        That differs from my past experience (in which
                        a proposed spec change is sufficient to move
                        forward with a launch), but if you want me to
                        build and test it all behind a flag before I
                        get the go ahead here that's fine.

                        ~ Ari Chivukula (Their/There/They're)


                        On Wed, Apr 19, 2023 at 11:32 AM Rick Byers
                        <rby...@chromium.org> wrote:

                            This makes a lot of sense to me. Normally
                            we prefer to approve shipping only after
                            spec PRs have landed, it looks like the PR
                            is blocked on some work from you including
                            landing the WPTs, right? I'd suggest you
                            get this all landed under a
                            status=experimental RuntimeEnabledFeature
                            and see if we can get the PR reviewed and
                            landed in short order before trying to
                            flip to status=stable, WDYT?

                            Rick

                            On Wed, Apr 5, 2023 at 2:46 PM 'Ari
                            Chivukula' via blink-dev
                            <blink-dev@chromium.org> wrote:

                                
https://github.com/w3ctag/design-reviews/issues/833

                                On Wednesday, April 5, 2023 at
                                11:27:22 AM UTC-4 Alex Russell wrote:

                                    Please file an issue with the TAG
                                    as an FYI and let us know when
                                    they respond. Their job is to help
                                    us drive coherence across the
                                    platform and spot architectural
                                    patterns. There might be other
                                    occurances of similar oversights
                                    that they can help us spot and fix.

                                    Best,

                                    Alex

                                    On Tuesday, March 28, 2023 at
                                    11:15:31 AM UTC-7 Ari Chivukula wrote:

                                        Contact emails

                                        ari...@chromium.org,
                                        mike...@chromium.org,
                                        yoav...@chromium.org


                                        Specification

                                        
https://github.com/w3c/navigation-timing/pull/188
                                        
<https://github.com/w3c/navigation-timing/pull/188>


                                        Summary

                                        Websites can indicate that a
                                        particular Client Hint
                                        
<https://wicg.github.io/client-hints-infrastructure/>is
                                        critical to the page by
                                        including it in a
                                        `Critical-CH` HTTP response
                                        header. Doing so will trigger
                                        a connection restart if the
                                        hint listed in the
                                        `Critical-CH` HTTP response
                                        header could be (but wasn’t)
                                        included in the HTTP request
                                        initially sent.This intent
                                        proposes adding `readonly
                                        attribute DOMHighResTimeStamp
                                        criticalCHRestart;` to the
                                        `PerformanceNavigationTiming`
                                        interface.


                                        Blink component

                                        Blink>PerformanceAPIs>NavigationTiming
                                        
<https://bugs.chromium.org/p/chromium/issues/list?q=component%3ABlink%3EPerformanceAPIs%3ENavigationTiming>


                                        Motivation

                                        It’s possible for websites to
                                        observe a change in the
                                        Navigation Timing
                                        
<https://github.com/w3c/navigation-timing>related
                                        to `responseStart` and
                                        `requestStart`, but there is
                                        no explicit timestamp that
                                        indicates when this occurred.
                                        Without that it’s harder for
                                        sites to detect the impact
                                        that the `Critical-CH` HTTP
                                        response header has on
                                        connections. Developers
                                        (including Akamai) are
                                        interested in using this to
                                        better understand the impact
                                        of User Agent Client Hints /
                                        User Agent Reduction.


                                        `criticalCHRestart` can only
                                        be set for top-level frames,
                                        as subresources and iframes
                                        ignore `Critical-CH` HTTP
                                        response headers.


                                        TAG review

                                        N/A, this is a small,
                                        incremental addition to an
                                        existing feature for
                                        convenience and the loaded
                                        website has other (less
                                        convenient) ways to detect a
                                        Critical-CH restart.


                                                Compatibility &
                                                Interoperability

                                        We would be the first to
                                        implement if approved.


                                        Gecko:
                                        
https://github.com/mozilla/standards-positions/issues/767
                                        
<https://github.com/mozilla/standards-positions/issues/767>


                                        WebKit:
                                        
https://github.com/WebKit/standards-positions/issues/156
                                        
<https://github.com/WebKit/standards-positions/issues/156>


                                        Web Developers: Requested by
                                        Akamai
                                        
<https://github.com/w3c/navigation-timing/issues/177>


                                        Is this feature fully tested
                                        by web-platform-tests?

                                        No, but it will be.


                                        Tracking bug

                                        https://crbug.com/1425910
                                        <https://crbug.com/1425910>


                                        Link to entry on the Chrome
                                        Platform Status

                                        
https://chromestatus.com/feature/5140317616996352
                                        
<https://chromestatus.com/feature/5140317616996352>


-- 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/0dacf540-c8be-4096-9ad1-bc7a99af630cn%40chromium.org
                                
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/0dacf540-c8be-4096-9ad1-bc7a99af630cn%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/CAGpy5DKthEYQEYGqoFyVWsp0xnm24QLEJfEoeZz9ZKB39PKtkg%40mail.gmail.com
        
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGpy5DKthEYQEYGqoFyVWsp0xnm24QLEJfEoeZz9ZKB39PKtkg%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/CAOMQ%2Bw8xuuVTFCSKCyvqu-_7mnR6xpdzqbKkVd8T48S30rhJsQ%40mail.gmail.com
    
<https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw8xuuVTFCSKCyvqu-_7mnR6xpdzqbKkVd8T48S30rhJsQ%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/CAARdPYfz2QM8hj_1GdYL%2BwGhey9HPjAAPcMwmJLUEkm697n2Sw%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAARdPYfz2QM8hj_1GdYL%2BwGhey9HPjAAPcMwmJLUEkm697n2Sw%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/75ec0c4b-5590-3872-4028-438543406a6f%40gmail.com.

Reply via email to